Skip to content

Fix docs pipeline so that it doesn't lint outside /doc

Marcel Amirault requested to merge fix-docs-pipeline-incremental-test into master

What does this MR do?

I think master is broken for an edge case, where changes to .md files outside of doc/ are being linted by accident, because I forgot to limit my wildcard filter. This changes *.md to doc/*.md to avoid accidentally linting things like changelogs/, etc.

Example of a failure: https://gitlab.com/gitlab-org/gitlab/-/jobs/718867042

Testing

Tested by adding a commit that changed a doc (broke it to make it fail linting) in /doc and one in /changelogs (which already should fail linting): https://gitlab.com/gitlab-org/gitlab/-/jobs/719695860

Only the /doc doc was linted, which is the correct behavior. After that, I pulled the commit changing the docs, as it's no longer needed.

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
Edited by Marcel Amirault

Merge request reports

Loading