Skip to content

Run full vale and markdownlint for docs pipeline changes

Marcel Amirault requested to merge update-docs-test-for-pipeline-changes into master

What does this MR do?

This tries to prevent broken masters caused by MRs like: !87981 (merged)

In this case:

The key details is in the first pipeline:

Merged results pipeline detected. Testing only the following files:
doc/raketasks/sidekiq_job_migration.md

When running merged results pipelines, we normally only lint the changed files because it speeds up the job 10x. In a few cases, we want to always run on ALL docs, such as when we change the linting configuration or script. The part we were missing was that we also want to lint ALL docs when we change the docs pipeline configuration too. So, this MR:

  • Changes the pipeline so that docs linting jobs run when the docs pipeline configuration is changed.
  • Changes the script so that ALL docs are linted when the pipeline configuration is changed.

Related issues

Checklist

Pre-merge

Consider the effect of the changes in this merge request on the following:

If new jobs are added:

This will help keep track of expected cost increases to the GitLab project average pipeline cost per merge request RPI

Post-merge

Merge request reports