Updated contribution documentation + Removed WPEngine logo by wimg · Pull Request #904 · PHPCompatibility/PHPCompatibility
Navigation Menu
- Notifications You must be signed in to change notification settings
- Fork 204
Conversation
- Updated contribution documentation to reflect change from master to develop
- Removed WPEngine logo (wasn't working anymore). Kept link.
jrfnl
left a comment
•
Loading
jrfnl
left a comment
•
Loading
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- I'm wondering if we may need to update the links for some of the badges in the Readme too to point explicitly to
developormaster? - Regarding projects using PHPCompatibility: I'm wondering if we should limit this to projects which function like a shell around PHPCompatibility, like the first three listed.
Listing external PHPCS standards which just use PHPCompatibility as part of their internal process /require-devseems like it may become a long, arbitrary and unmanageable list.
More external projects / shells:
- PHPCompatibility Checker WordPress plugin
- WordPress Tide project
- The fact that PHPStorm contains build-in support for PHPCompatibility
- I'm wondering if we may need to update the links for some of the badges in the Readme too to point explicitly to
developormaster?
Changed that. Consistency is indeed key there.
- Regarding projects using PHPCompatibility: I'm wondering if we should limit this to projects which function like a shell around PHPCompatibility, like the first three listed.
Updated it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters