- Issue created by @jonathan1055
- First commit to issue fork.
- 🇳🇿New Zealand quietone
I added this back in but the changes here are deployed to the 'real' URL. Did I make a mistake?
# Temporary change to rebuild the docs in MR without a commit to default branch. pages: rules: - !reference [ .skip-pages-rule ] - exists: - mkdocs.yml when: on_success
I also noticed a bunch of information errors at https://git.drupalcode.org/project/coding_standards/-/jobs/5613053#L144. So there is more cleanup to do. How about expanding the scope of this to include that?
- 🇬🇧United Kingdom jonathan1055
I added this back in but the changes here are deployed to the 'real' URL. Did I make a mistake?
That is the expected behavior when those lines are added in a MR and the pipeine is run by a maintainer. Unfortunately we don't have any solution for this yet, and its not satisfactory at all. If I were to submit a pipeline on this MR the pages would be built to a different url, and the live site would not be affected until the MR was committed. This is the same as what I noted in the issue summary on 📌 Formatting fixes Active . I have updated Gitlab Templates issue 📌 Only rebuild documentation pages when source has changed Active so hopefully we can make some progress there.
I also noticed a bunch of information errors
Well spotted. I wonder if those could be highlighted better in the log, or somehow we draw attention to them. Yes, definitely fix in this MR (I think you have done some already)
- 🇬🇧United Kingdom jonathan1055
Just checked, and confirm that all of the redundant links have been removed, and also you have fixed all of the info/warnings about missing urls. So this is RTBC from me.
-
quietone →
committed e45d2cde on main
Issue #3530937 by quietone, jonathan1055: Remove redundant secondary...
-
quietone →
committed e45d2cde on main