Skip to content

msvc build improvements#3359

Merged
Jens-G merged 1 commit intoapache:masterfrom
Jens-G:fix-pr-msvc
Mar 22, 2026
Merged

msvc build improvements#3359
Jens-G merged 1 commit intoapache:masterfrom
Jens-G:fix-pr-msvc

Conversation

@Jens-G
Copy link
Member

@Jens-G Jens-G commented Mar 22, 2026

  • add concurrency group to cancel redundant runs
  • pin GitHub action versions to full SHA
  • add error handling and timeouts to Docker operations
  • Classify pull failures: 404=rebuild, auth/registry errors=fail loud
  • Add timeouts: pull(10m), build(60m), push(20m), test(120m)
  • restrict packages:write permission to non-PR runs
  • Remove packages:write from workflow-level permissions
  • Add step-level permissions to GHCR login step
  • Add condition to skip login for pull requests (including forks)
  • Fork PRs now have minimal permissions (contents:read only)

Generated-by: OpenCode big-pickle

- add concurrency group to cancel redundant runs
- pin GitHub action versions to full SHA
- add error handling and timeouts to Docker operations
- Classify pull failures: 404=rebuild, auth/registry errors=fail loud
- Add timeouts: pull(10m), build(60m), push(20m), test(120m)
- restrict packages:write permission to non-PR runs
- Remove packages:write from workflow-level permissions
- Add step-level permissions to GHCR login step
- Add condition to skip login for pull requests (including forks)
- Fork PRs now have minimal permissions (contents:read only)

Generated-by: OpenCode big-pickle
@mergeable mergeable bot added the github_actions Pull requests that update GitHub Actions code label Mar 22, 2026
@Jens-G Jens-G merged commit f8622a7 into apache:master Mar 22, 2026
79 of 80 checks passed
@Jens-G Jens-G deleted the fix-pr-msvc branch March 22, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant