Skip to content

Conversation

@monzuralam
Copy link

Rename plugin.php to plugin-check.php

As per the plugin development Best Practices Folder structure recommended, the plugin's main file name should be plugin folder name.php. Plugin Check is our plugin name, So, plugin.php file should be plugin-check.php.

I also provide a reference from WordPress Plugin Review Team:

The main file of the plugin has a name that does not follow the convention.

We expect the main plugin file (the file containing the plugin headers) to have the same name as the plugin folder, which is also the same name as the slug / permalink of the plugin.

For example, if your plugin slug is ecpt-social-manager we expect your main plugin filename to be ecpt-social-manager.php

Note that using some common names as the filename for the main plugin file can lead to issues in some configurations.

Please check out our tips on how to structure files and folders in a plugin.

The main file of this plugin is named plugin.php

@github-actions
Copy link

github-actions bot commented Aug 4, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: monzuralam <monzuralam@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@swissspidy
Copy link
Member

We know :-)

Thanks for your contribution, but we can‘t do this for backward compatibility reasons, as this would cause the plugin to deactivate automatically for all users.

@swissspidy swissspidy closed this Aug 4, 2024
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.

2 participants