Skip to content

Explicitly call out that contributions do not require product approval

Joshua Lambert requested to merge joshlambert-master-patch-88402 into master

What does this MR do?

This MR specifically notes that Product approval is not required for contributions. Currently it is not part of the Definition of Done, but some could have understood it to be implied.

This is being added now because some parts of GitLab specific, but also potentially contributions from the wider community, needed to engage with Product and receive approval prior to contributing a change/feature.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • 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
Edited by Joshua Lambert

Merge request reports

Loading