Skip to content

Third Try Dependabot#150

Merged
wallstop merged 2 commits intomasterfrom
dev/wallstop/fix-ci-3
Jan 28, 2026
Merged

Third Try Dependabot#150
wallstop merged 2 commits intomasterfrom
dev/wallstop/fix-ci-3

Conversation

@wallstop
Copy link
Copy Markdown
Owner

Description

Maybe this will fix dependabot?

Related Issue

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactor (code change that neither fixes a bug nor adds a feature)

Checklist

  • All tests pass locally
  • Code is properly formatted
  • I have added tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have updated the CHANGELOG
  • My changes do not introduce breaking changes, or breaking changes are documented

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request addresses issues with git add --renormalize commands failing in GitHub Actions workflows when file patterns don't match any files (exit code 128). The PR introduces a validation script, updates all workflows to use a safer per-extension loop pattern, and documents the required patterns.

Changes:

  • Added a validation script to detect problematic git renormalize patterns
  • Refactored all workflow files to use per-extension loops with existence checks
  • Added comprehensive test coverage for the validation logic
  • Updated documentation to describe the required pattern and validation process

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
scripts/validate-workflows.js New validation script that detects forbidden multi-pattern renormalize commands and missing existence checks
scripts/validate-workflows.js.meta Unity meta file for the validation script
scripts/__tests__/validate-workflows.test.js Comprehensive test suite for validation logic
scripts/__tests__/validate-workflows.test.js.meta Unity meta file for the test file
.github/workflows/prettier-autofix.yml Updated 3 renormalize commands to use per-extension loop pattern
.github/workflows/format-on-demand.yml Updated 4 renormalize commands to use per-extension loop pattern
.github/workflows/csharpier-autofix.yml Updated 3 renormalize commands to use per-extension loop pattern
.github/workflows/actionlint.yml Added validation script execution to CI pipeline
.llm/skills/github-actions/git-renormalize-patterns.md Documented required per-extension loop pattern and forbidden patterns, added verification section
.llm/skills/index.md Updated line count for git-renormalize-patterns skill (266 → 348 lines)
.llm/context.md Added guidance to use per-extension loops with existence checks

Comment thread .llm/context.md Outdated
Comment thread .llm/skills/github-actions/git-renormalize-patterns.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.

@wallstop wallstop merged commit bd22430 into master Jan 28, 2026
19 checks passed
@wallstop wallstop deleted the dev/wallstop/fix-ci-3 branch January 28, 2026 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants