Improve readme and grunt tasks#130
Conversation
danielbachhuber
left a comment
There was a problem hiding this comment.
Couple minor questions
| wp-cli.local.yml | ||
| tests | ||
| /vendor/lcache/lcache/tests | ||
| node_modules |
There was a problem hiding this comment.
Why'd you remove these?
There was a problem hiding this comment.
@danielbachhuber I added trailing slashes to indicate directories for clarity. The others are covered by *.yml and *.xml. See the full file below.
https://github.com/fjarrett/wp-lcache/blob/2a58d664dc0a4211ea159599f09608adf5237c60/.distignore
There was a problem hiding this comment.
I also rearranged them in order of files (a-z) then directories (a-z)
| Tags: cache, plugin | ||
| Tags: apc, apcu, cache, caching, lcache, object-cache, performance | ||
| Requires at least: 4.3 | ||
| Tested up to: 4.7.4 |
There was a problem hiding this comment.
@danielbachhuber Because WordPress ignores minor releases in this field so you aren't required to bump your versions at every point release, making it just added verboseness.
https://plugins.svn.wordpress.org/jetpack/trunk/readme.txt
https://plugins.svn.wordpress.org/woocommerce/trunk/readme.txt
https://plugins.svn.wordpress.org/wordpress-seo/trunk/readme.txt
There was a problem hiding this comment.
Because WordPress ignores minor releases in this field so you aren't required to bump your versions at every point release, making it just added verboseness.
Makes sense.
Replacement for #125
package.jsonmatchdep.distignoreReadme preview here: https://github.com/fjarrett/wp-lcache/tree/improved-readme#wp-lcache