← Back to issue list

docs: add docs-linkcheck to the commands to test

View original Github issue

Metadata

Project
starbase
Number
#630
Type
pull request
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Test PR verifying starflow's docs-only path filter; not intended to merge. Closed after testing purpose was served.

Resolution: closed resolved

Reason: This was a test PR (stacked on #627) to verify CI path filtering behavior. It was never intended to be merged and was closed after its testing purpose was served.

Evaluated by: mr-cal-bot

No scores available.

Issue body

## Purpose Docs-only filter test, stacked on canonical/starbase#627. Not intended to merge. This branch changes `docs/README.md` and nothing else. ## What it tests Whether starflow's paths filter classifies this change as documentation only, so the right jobs run: | Job | Expected | Why | |---|---|---| | `filter` | runs | always | | `files` (`make -k lint-code`) | **skipped** | `docs/**` and `**/*.md` are excluded from `source-files` | | `docs` (`make -k docs-lint`) | runs | `docs-files` matches `docs/**` | | `linkcheck` (`make -k docs-linkcheck`) | runs | `docs-files` matches `docs/**`, then fails on the missing target | This is the "yes-docs, no-code" case. It also confirms the `linkcheck` job is gated on `docs-files` alone, so it starts even when there is no source change. `docs-lint` must pass. `linkcheck` is expected to fail with "No rule to make target 'docs-linkcheck'", because this stack deliberately does not include the starbase change from canonical/starbase#625. Note the diff adds `make docs-linkcheck` to a list of commands, but the target does not exist on this branch. That is deliberate: it documents the ordering problem this stack exists to measure. ## Stacked on this Source-plus-docs filter test: canonical/starbase#633 ## Related - canonical/starbase#625 - canonical/starbase#627 - canonical/starflow#199

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — closed resolved Test PR verifying starflow's docs-only path filter; not intended to merge. Closed after testing purpose was served.

Update history

Date Change
closed
updated
created

Related work

  • Superseded By: canonical/starbase#627 (confidence 90%)

    PR #630 was stacked on #627, which served the same testing purpose for the split lint workflow.

  • Related To: canonical/starbase#633 (confidence 85%)

    PR #633 was stacked on #630 as the source-plus-docs filter test.

  • Related To: canonical/starbase#625 (confidence 80%)

    PR #625 added the docs-linkcheck target that #630 was testing against.

Related issues

Issue Project State Summary Similarity
#628 docs: add docs-linkcheck to the commands to test starbase closed Test PR to validate CI path filtering for docs-only changes. Closed without merging after confirming the filter correctly skips lint-code and runs docs jobs.
83%
#2972 ci: test docs and linkcheck jobs in starflow lint workflow charmcraft closed Draft test PR to verify starflow lint workflow docs and linkcheck jobs. Closed without merging as it was only for integration testing.
82%
#398 docs: add linkcheck to doc checks starbase merged Merged in canonical/starbase#398 as d240af2, adding Sphinx linkcheck to documentation checks after the scope was narrowed to linkcheck only.
77%
#980 build(docs): add starbase linkcheck filters rockcraft merged Merged PR #980 as commit 1e5304ea, adding Starbase linkcheck filters to the docs build; all CI checks passed.
76%
#1771 [DO NOT MERGE] Test starflow PR 201 craft-parts closed Automated test PR to verify CI workflows for starflow PR #201. Closed after verification completed.
73%
#667 [DO NOT MERGE] Test starflow PR 201 fetch-service closed Automated test PR to verify downstream CI for starflow PR #201. Closed after verification without merging, as intended.
73%
#500 [DO NOT MERGE] Test starflow PR 201 imagecraft closed Test PR to verify starflow PR #201 CI workflows. Closed without merging after verification.
73%
#503 [DO NOT MERGE] Test starflow PR 201 craft-cli closed Automated test PR to verify starflow PR #201 CI workflows. Closed without merging as intended.
72%
#1390 [DO NOT MERGE] Test starflow PR 201 rockcraft closed Automated test PR for starflow PR #201 CI workflows. Closed without merging after verification, per its "DO NOT MERGE" intent.
72%
#207 [DO NOT MERGE] Test starflow PR 201 debcraft closed Closed after verifying downstream CI workflows for canonical/starflow#201. Intended as a test draft and not merged.
72%