Skip to content

Remove hyperbolic tone from SQL guideline on pluck

Dylan Griffith requested to merge remove-hyperbole-from-database-docs-pluck into master

What does this MR do?

The statement "Never use" is already sufficiently clear and does not need to be prefixed with another level of hyperbole. It makes me uncomfortable when I link people to this line of the docs since it feels like such a hostile way to give someone feedback. We can remove the passion from these docs and still convey the same importance.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports