-
Notifications
You must be signed in to change notification settings - Fork 107
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
Define name for the performance plugin #1
Comments
Name: Core Performance Tests |
Probably best to reflect somewhat on the test or experimental nature of it, because you probably don't want people installing it on sites that are meant to be 100% stable. Perf-experiments, or something like that? I suck at naming, but something that reflects that it's for testing only. |
|
We should also mention this in the repo README/docs. The experimental nature of the plugin, and the fact that it shouldn't be used as a permanent solution on sites. |
There are two very-very separate parts of WordPress performance.
Please take this into consideration. |
Just throwing some ideas, While some modules will be for testing, it is also important to keep in mind that it should be developed in a reliable way and not only deployed on throw away sites (like any feature plugins would, and so is Gutenberg). For that reason it would be preferable not to over index on the experimental nature (IMO) which could lead to not getting the minimal user base adoption we are looking for. |
|
I'd like to see a more specific name, I still like my suggestion |
@ecotechie - I like the How about |
Core says nothing to "average" users, who should be installing this too. I like just |
I like |
+1 for |
+1 for |
+1 for |
So we do expect/hope production sites install this plugin? And "average" users will install this on their live sites. I thought it was for testing purposes only. I then assume there will be a warning that it may break things? |
The concept of this plugin is similar to that of Gutenberg. So no, I don't think we should have warnings etc. All plugins "may break things", nothing is ever 100% feature-complete and bug-free. |
Not trying to be a pain, but if this plugin is meant as a lab or for tests, it will break "things". Same with the Gutenberg plugin. I help maintain over 600 sites and if we ever see Gutenberg installed, we recommend removing it. Since it can be unstable and features can be removed. I know the nature of plugins is such that the code will sometimes cause unintended effects. However, a plugin dev that makes a well intentioned mistake and an essentially beta testing plugin are very different. I'll leave it at that and sorry to drag this out, but I feel users should be made aware when plugins are not meant for production sites. |
…ure-row Show notice with picture element field
As outlined in the original proposal doc, the performance plugin doesn't have a name yet. Some ideas were previously discussed in the doc and on Slack, but we can follow up here.
Once a name (and slug!) have been decided for, we need to replace the following in the scaffolded codebase:
TODO-SLUG
with plugin slug/textdomainTODO-NAME
with (human-friendly) plugin nametodo_snakecaseslug
with snake_case version of the plugin slug (e.g. prefix for functions, options, etc.)TODO_CONSTANTSLUG
with CONSTANT_CASE version of the plugin slug (e.g. prefix for constants)The text was updated successfully, but these errors were encountered: