Skip to content

fix(ci): drop renovate annotation test and resolve dprint/yamlfmt drift#166

Merged
DevSecNinja merged 3 commits into
mainfrom
copilot/fix-ci-pipeline-failures
Jun 20, 2026
Merged

fix(ci): drop renovate annotation test and resolve dprint/yamlfmt drift#166
DevSecNinja merged 3 commits into
mainfrom
copilot/fix-ci-pipeline-failures

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

CI was failing on test_repo_automation_workflows_are_onboarded and on the org lint workflow's dprint and yamlfmt jobs.

Following the move to DevSecNinja/.github#192, the per-caller # renovate annotation on reusable-workflow callers is no longer required, so the onboarding test that enforced it is removed rather than the annotation being restored.

Changes

  • tests/test_repo_automation.py: removed the # renovate: datasource=github-tags depName=DevSecNinja/.github assertion from test_repo_automation_workflows_are_onboarded.
  • .github/workflows/config-sync.yml: reverted the previously added # renovate annotation, keeping the caller as Renovate bumped it.
  • .github/workflows/pages.yml: converted the test-command input to a literal block scalar (|-) so the long command is preserved verbatim. This fixes the lint / yamlfmt failure (yamlfmt no longer reflows the >120-column line) without breaking test_pages_workflow_runs_after_main_updates, which asserts the raw command substring. The command string passed to the reusable workflow is unchanged.
  • docs/architecture.md, docs/github-actions.md, docs/requirements.md, docs/testing.md: reformatted markdown table alignment to satisfy the lint / dprint job. These are whitespace-only alignment changes with no content modifications.

All 94 tests, check_format, lint, yamlfmt, and dprint pass locally.

Copilot AI changed the title [WIP] Fix CI pipeline failures fix(ci): restore renovate annotation on config-sync workflow Jun 19, 2026
Copilot AI requested a review from DevSecNinja June 19, 2026 19:46
@DevSecNinja DevSecNinja marked this pull request as ready for review June 20, 2026 07:45
@github-actions github-actions Bot added the area/github GitHub Actions, Renovate, labels, and repository automation label Jun 20, 2026
@DevSecNinja

Copy link
Copy Markdown
Owner

@copilot dprint and yamlfmt are still failing. Also, can you get rid of the annotation test instead because: DevSecNinja/.github#192

Copilot AI commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot dprint and yamlfmt are still failing. Also, can you get rid of the annotation test instead because: DevSecNinja/.github#192

Done in the latest commit:

  • Removed the renovate-annotation assertion from test_repo_automation_workflows_are_onboarded and reverted the config-sync.yml annotation change (per docs(renovate): drop redundant uses-line annotations in onboarding guide and workflows .github#192).
  • lint / yamlfmt: converted the pages.yml test-command to a literal block scalar so the long line is preserved verbatim — this fixes yamlfmt without breaking test_pages_workflow_runs_after_main_updates.
  • lint / dprint: reformatted markdown table alignment in docs/architecture.md, docs/github-actions.md, docs/requirements.md, and docs/testing.md.

All 94 tests, check_format, lint, yamlfmt, and dprint pass locally.

Copilot AI changed the title fix(ci): restore renovate annotation on config-sync workflow fix(ci): drop renovate annotation test and resolve dprint/yamlfmt drift Jun 20, 2026
@github-actions github-actions Bot added the area/docs Documentation and ADRs label Jun 20, 2026
@DevSecNinja DevSecNinja merged commit 0201a3e into main Jun 20, 2026
16 checks passed
@DevSecNinja DevSecNinja deleted the copilot/fix-ci-pipeline-failures branch June 20, 2026 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Documentation and ADRs area/github GitHub Actions, Renovate, labels, and repository automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants