Skip to content

fix(ci): preserve tagged wrapper executable mode - #20

Merged
bglglzd merged 1 commit into
mainfrom
fix/release-wrapper-mode
Jul 13, 2026
Merged

fix(ci): preserve tagged wrapper executable mode#20
bglglzd merged 1 commit into
mainfrom
fix/release-wrapper-mode

Conversation

@bglglzd

@bglglzd bglglzd commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • validate gradlew.bat executable state against its exact Git index mode
  • accept only repository modes 100644 or 100755
  • retain the byte-level CRLF/LF-only content comparison for older tags

Root cause

The prior validator correctly detected the only dirty path but assumed a Windows batch file must be non-executable. The tagged HeliBoard-derived source intentionally records gradlew.bat as 100755; GitHub checkout preserved that mode.

Verification

  • actionlint .github/workflows/release.yml
  • python scripts/security_source_scan.py .
  • git diff --check

Signed-off-by: bglglzd <248948303+bglglzd@users.noreply.github.com>
@bglglzd
bglglzd enabled auto-merge (squash) July 13, 2026 15:55
@bglglzd
bglglzd merged commit 2fec923 into main Jul 13, 2026
16 checks passed
@bglglzd
bglglzd deleted the fix/release-wrapper-mode branch July 13, 2026 16:05
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.

1 participant