-
Notifications
You must be signed in to change notification settings - Fork 46
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
Add WordPress plugin comment header #46
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. Happy to add this, pending a few changes.
plugin.php
Outdated
* Cavalcade! | ||
* Plugin Name: Cavalcade | ||
* Plugin URI: https://github.com/humanmade/Cavalcade | ||
* Description: Scalable job system, designed as a drop-in replacement for wp_cron |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should copy from the existing description:
A better wp-cron. Horizontally scalable, works perfectly with multisite.
plugin.php
Outdated
* Plugin Name: Cavalcade | ||
* Plugin URI: https://github.com/humanmade/Cavalcade | ||
* Description: Scalable job system, designed as a drop-in replacement for wp_cron | ||
* Version: 0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unless it's required, I'd rather leave off the version until we start versioning Cavalcade.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll check if it works even without the version. I'll use dev-master
if it's mandatory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The readme says that in order to use this plugin I need to add extra
mu-plugins/cavalcade.php
.I usually have so many mu-plugins that this practice gets really hard to maintain and normally I don't add extra includes for each mu-plugin but instead I use bedrock-autoloader.php.
This can load all plugins from subfolders inside mu-plugins folder. But the problem here is that it doesn't work if the plugin in subfolder doesn't have correct comment headers in the start of the file.
This pull request fixes this problem and allows installation into subfolder without extra steps.
More info: https://roots.io/bedrock/docs/mu-plugins-autoloader/