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

fix: Remove Plugin Check Errors & Warnings 🐞 #11

Closed
wants to merge 1 commit into from

Conversation

ashutoshgautams
Copy link
Contributor

Plugin Check Compliance Updates

What

  • Removed the Update URI header from the main plugin file
  • Reduced the number of tags in readme.txt from 7 to 5

Why

To comply with WordPress.org plugin guidelines and improve the plugin's compatibility score in Plugin Check.

Related Issue(s):

How

  1. Removed the Update URI: and GitHub Plugin URI: headers from snapwp-helper.php
  2. In readme.txt, reduced tags to: decoupled, headless, GraphQL, WPGraphQL, JAMStack

Testing Instructions

  1. Create a new release zip file with these changes
  2. Install the Plugin Check plugin on a test WordPress site
  3. Upload and activate the new SnapWP Helper plugin zip
  4. Run Plugin Check on SnapWP Helper
  5. Verify that the "Use of the Update URI header is not allowed" error is resolved
  6. Verify that the "too many tags" warning is resolved

Screenshots

[Include screenshots of the successful Plugin Check results here]

Additional Info

These changes address the main issues flagged in the initial Plugin Check audit. Further audits may be necessary to ensure full compliance.

Checklist

  • I have read the Contribution Guidelines.
  • My code is tested to the best of my abilities.
  • My code passes all lints (PHPCS, PHPStan, ESLint, etc).
  • My code has detailed inline documentation.
  • I have added unit tests to verify the code works as intended.
  • I have updated the project documentation accordingly.

- Use of the Update URI header is not allowed
- Reduced the number of tags from 7 to 5 in readme.txt
@ashutoshgautams ashutoshgautams changed the title fix: Remove Plugin Check errors & warnings fix: Remove Plugin Check Errors & Warnings 🐞 Oct 5, 2024
@ashutoshgautams ashutoshgautams self-assigned this Oct 5, 2024
@ashutoshgautams
Copy link
Contributor Author

image

All the errors & warnings were removed @justlevine.

@justlevine
Copy link
Collaborator

#8 (comment)

@justlevine justlevine closed this Oct 5, 2024
@justlevine justlevine deleted the fix/plugin-check-errors branch January 9, 2025 22:30
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.

Chore: Perform PCP audit
2 participants