Skip to content

Fix "pages not in nav" command

Achilleas Pipinellis - OOO until November 25th (parental leave) requested to merge axil-fix-page-not-nav-script into main

What does this MR do and why?

The current invocation of make check-pages-not-in-nav fails because we also run install-nodejs-dependencies which has some Yarn output that jq doesn't like. It seems this behavior changed when we updated to Yarn 4.

Additionally, an exclusion for /user/application_security/dast/browser/checks/ has been added to the script to exclude those pages from the check. Seems the old location was changed.

Reported in technical-writing#985 (comment 1778462638)

How to set up and validate locally

  1. Configure a local GitLab Docs environment: https://gitlab.com/gitlab-org/gitlab-docs/-/blob/main/doc/setup.md.
  2. Run the following and make sure there's no error: 
    make check-pages-not-in-nav
make check-pages-not-in-nav | jq '.[] | .url'

Merge request acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Achilleas Pipinellis - OOO until November 25th (parental leave)

Merge request reports