search

WPTT / WPThemeReview

PHP_CodeSniffer rules (sniffs) to enforce WordPress theme review coding conventions
MIT License
207 stars 38 forks source link

Minor prerelease updates #225

Closed dingo-d closed 5 years ago

dingo-d commented 5 years ago

I've updated the changelog and replaced old links of WPCS github repository with the new one.

Once this is merged I can tag and release 0.2.0 version 🎉

dingo-d commented 5 years ago

Thanks for the review, I've commited the changes.

It contains a lot of details, some of which is not really interesting or even relevant to the users of the repo. So my question is: who is your target audience for the changelog ?

The target audience should be the users of the ruleset, so I've removed some extra information. Most of the sniffs have extra information in their docblocks, and I plan on adding the documentation like in the WPCS so that extra information can be added for every sniff.

What is the logic for the order of the entries in each section?

I went to the commit list and just went commit by commit from the last commit to the first one. So the order logic is chronological.

I've also closed the issues or moved them to the next release, so we're good there :+1: