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

Update output when dbDelta is found in the codebase #479

Closed
BrookeDot opened this issue Jun 25, 2020 · 4 comments · Fixed by #692
Closed

Update output when dbDelta is found in the codebase #479

BrookeDot opened this issue Jun 25, 2020 · 4 comments · Fixed by #692

Comments

@BrookeDot
Copy link
Contributor

What problem would the enhancement address for VIP?

When a codebase is scanned and dbDelta is found the following error is output:

All database modifications have to approved by the WordPress.com VIP team.

https://github.com/Automattic/VIP-Coding-Standards/blob/master/WordPressVIPMinimum/Sniffs/Functions/RestrictedFunctionsSniff.php#L94

While this was previously true on WordPress.com we no longer require the VIP team to approve all database changes at every code review level.

Describe the solution you'd like

As this can still cause performance problems, we should continue to flag this as a possible issue. However, instead of stating that the VIP team needs to approve the change, we could link to documentation of some things to check or other helpful tips.

@GaryJones
Copy link
Contributor

@BrookeDot Did you fancy doing a PR that changes this (in the the WordPress-VIP-Go ruleset) from Error to Warning, and adjusts the message wording?

@BrookeDot
Copy link
Contributor Author

Happy to submit a PR for the text change, but before I do I think we need to have a few more internal discussions around what the expectations should be for dbDelta on VIP Go.

@rebeccahum
Copy link
Contributor

After some internal discussion, it might be good to rid this warning because the usefulness of it is marginal on VIP Go.

@rebeccahum rebeccahum added this to the 2.3.3 milestone Jun 24, 2021
BrookeDot added a commit that referenced this issue Jun 24, 2021
This removes the dbDelta message which previously stated to contact
WordPress.com VIP support for review. Fixes #479.
@BrookeDot
Copy link
Contributor Author

Opened a PR ( #692) to remove the rule.

@rebeccahum rebeccahum removed this from the 2.3.3 milestone Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants