Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collect all Composer-managed plugins together #24

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

edjeavons
Copy link
Member

This is an idea that may need a little discussion. It shouldn't break anything (I have done a limited test) so it's a matter of whether or not we like the concept.

What's Changed

The change groups all of the website's Composer based plugins together on the filesystem by prefixing each with an underscore as per the screenshot below.

CleanShot 2023-04-15 at 17 14 24

Reasoning

  1. Our custom plugins (or third party ones what are not in Composer) are far easier to focus on when we are browsing the file system
  2. When doing website updates it's clearer which plugins we are responsible for updating manually vs leaving up to Composer
  3. All plugins beginning with _ will be automatically ignored by Git, allowing all others to be version-controlled without having to manually whitelist them each time.

Potential issues

Any plugin developer presuming to know their plugin's directory name and hardcoding it into their PHP will be caught out. To the best of my knowledge WP plugin directory names can be whatever a user wants though.

To assist with file system & repo management
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant