Skip to content

adds documentation for sniff WordPress.PHP.DiscouragedPHPFunctions#2494

Open
tikifez wants to merge 1 commit into
WordPress:developfrom
tikifez:docs/WordPress.PHP.DiscouragedPHPFunctions
Open

adds documentation for sniff WordPress.PHP.DiscouragedPHPFunctions#2494
tikifez wants to merge 1 commit into
WordPress:developfrom
tikifez:docs/WordPress.PHP.DiscouragedPHPFunctions

Conversation

@tikifez

@tikifez tikifez commented Sep 17, 2024

Copy link
Copy Markdown
Contributor

Related to #1722

@rodrigoprimo rodrigoprimo left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

@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

tikifez commented Feb 4, 2025

Copy link
Copy Markdown
Contributor Author

@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!

@rodrigoprimo

Copy link
Copy Markdown
Contributor

@tikifez, please let us know within a week if you are still interested in finishing this PR. If we don't hear back from you, we will presume you don't have time, and we will see if we can find someone else to take over and finish it. Thanks for your work so far!

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