Skip to content

Conversation

@ashfame
Copy link
Member

@ashfame ashfame commented Feb 13, 2023

Need to do this because we are using sanitize_url() in #184 which has been marked as un-deprecated but a new PHPCS release isn't out

Fixed in PHPCS but would only be released in PHPCS v3

@ashfame ashfame requested a review from psrpinto February 13, 2023 13:51
@ashfame ashfame self-assigned this Feb 13, 2023
Copy link
Member

@psrpinto psrpinto left a comment

Choose a reason for hiding this comment

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

👍 Should we create a ticket to upgrade to new wpcs version once it's released?

need to do this because we are using sanitize_url() which has been marked as undeprecated but a new PHPCS release isn't out
@ashfame ashfame force-pushed the phpcs_need_latest_develop branch from 4560074 to 056fe51 Compare February 13, 2023 14:09
@ashfame
Copy link
Member Author

ashfame commented Feb 13, 2023

I don't think we need to track that in an issue. Would get to update it eventually the next time need arises :)

@ashfame ashfame merged commit b2f2281 into main Feb 13, 2023
@ashfame ashfame deleted the phpcs_need_latest_develop branch February 13, 2023 14:12
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.

3 participants