Fix Copilot setup validation job by resolving workflow yamllint errors#1053
Conversation
|
@Bryan-Roe Unfortunately, I hit an error while trying to use the custom Copilot setup steps configured for this repository and had to close this PR. The error I am seeing is: Once you or someone with the necessary access fixes the problem, please unassign and then reassign issue #0 to me and I'll retry. Thanks! |
Security Policy Alert: Harden Runner Policy ViolationThis workflow run has been blocked by StepSecurity's Harden Runner policy. This workflow run was blocked because one or more jobs are missing the Harden-Runner action, or it is not properly configured, which can introduce agent-bypass risk. Jobs not protected by Harden Runner:
Update the affected jobs so Harden Runner is initialized before any other action or command runs. For more information, see StepSecurity's Harden Runner documentation. |
Security Policy Alert: Harden Runner Policy ViolationThis workflow run has been blocked by StepSecurity's Harden Runner policy. This workflow run was blocked because one or more jobs are missing the Harden-Runner action, or it is not properly configured, which can introduce agent-bypass risk. Jobs not protected by Harden Runner:
Update the affected jobs so Harden Runner is initialized before any other action or command runs. For more information, see StepSecurity's Harden Runner documentation. |
|
@Bryan-Roe Thanks for sending me some feedback. Unfortunately, I hit an error while trying to use the custom Copilot setup steps configured for this repository. The error I am seeing is: Once you or someone with the necessary access fixes the problem, please let me know in a comment and I'll try again. Thanks! |
Signed-off-by: Bryan <74067792+Bryan-Roe@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Fixes yamllint failures blocking Copilot setup validation.
Changes:
- Corrects YAML indentation and whitespace across six workflows.
- Adds focused workflow-format regression tests.
- One OSSAR regression test does not detect the original formatting errors.
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/code-coverage.yml |
Removes excess blank lines. |
.github/workflows/dataset-integrity.yml |
Corrects egress-policy indentation. |
.github/workflows/github_workflows_autofix-pr.yml |
Adds EOF newline. |
.github/workflows/github_workflows_validate-workflows.yml |
Adds EOF newline. |
.github/workflows/ossar.yml |
Corrects OSSAR comment indentation. |
.github/workflows/summary.yml |
Removes trailing whitespace. |
tests/test_workflow_yamllint_regressions.py |
Adds formatting regression coverage. |
| assert triggers["push"]["branches"] == ["main"] | ||
| assert triggers["pull_request"]["branches"] == ["main"] | ||
| assert steps[0]["name"] == "Checkout repository" |
🔐 CodeQL — Open Alerts on this PR
Copilot Autofix suggestions (if enabled) appear as inline review comments on the affected lines. |
The failing “Validate Copilot setup files & linters” job was caused by
yamllinthard errors across multiple workflow files (indentation, EOF newline, trailing whitespace, and excess blank lines). This PR applies targeted YAML-only corrections and adds focused regression coverage to prevent recurrence.Root cause remediation (workflow formatting errors)
permissionsincode-coverage.yml.with:block indentation foregress-policyin threedataset-integrity.ymljobs.github_workflows_autofix-pr.ymlgithub_workflows_validate-workflows.ymlossar.ymlYAML style/structure issues:["main"])stepslist indentation.summary.yml.Regression coverage
tests/test_workflow_yamllint_regressions.pyto lock in the exact failure patterns:permissionsincode-coverage.ymlwith/egress-policyindentation pattern indataset-integrity.ymlossar.ymltrigger/steps structure validitysummary.yml