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

adds documentation for sniff WordPress.PHP.DiscouragedPHPFunctions #2494

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

Conversation

tikifez
Copy link

@tikifez tikifez commented Sep 17, 2024

Related to #1722

Copy link
Collaborator

@rodrigoprimo rodrigoprimo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution, @tikifez! Before I review this PR in detail, could you please fix the same points, where applicable, that were discussed in #2487?

Quickly skimming through the XML file, I can see some issues with indentation, lack of <em> tags, and no newline at the end of the file.

@rodrigoprimo
Copy link
Collaborator

@tikifez, I was just wondering if you'll have a chance to finish this off in the near future. It would be great if this PR could be included in the next WPCS release.

If you haven't got time or lost interest, please let us know and we'll see if we can find someone to take over. Thanks!

@tikifez
Copy link
Author

tikifez commented Feb 4, 2025

@tikifez, I was just wondering if you'll have a chance to finish this off in the near future. It would be great if this PR could be included in the next WPCS release.

If you haven't got time or lost interest, please let us know and we'll see if we can find someone to take over. Thanks!

I'll get it done this week! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants