Skip to content

[Sync] Update project files from source repository (6bf8447)#121

Merged
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-mrz-tools-20260304-151742-6bf8447
Mar 4, 2026
Merged

[Sync] Update project files from source repository (6bf8447)#121
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-mrz-tools-20260304-151742-6bf8447

Conversation

@mrz1836
Copy link
Owner

@mrz1836 mrz1836 commented Mar 4, 2026

What Changed

  • Added github-token input parameter to .github/actions/setup-go-with-cache/action.yml and .github/actions/warm-cache/action.yml for private module authentication
  • Added new conditional step "🔐 Configure private module authentication" in setup-go-with-cache action that runs only when both github-token and GOPRIVATE environment variable are set
  • The authentication step configures git to use token-based HTTPS authentication for GitHub URLs and sets GONOSUMDB to match GOPRIVATE if not already set
  • Added github-token: ${{ secrets.GITHUB_TOKEN }} parameter to setup-go-with-cache and warm-cache action calls across multiple workflow files (fortress-code-quality.yml, fortress-benchmarks.yml, fortress-coverage.yml, fortress-release.yml, fortress-pre-commit.yml, fortress-test-fuzz.yml, fortress-test-matrix.yml, fortress-setup-config.yml, fortress-security-scans.yml, fortress-warm-cache.yml, fortress-test-suite.yml, fortress.yml)

Why It Was Necessary

  • Enable support for private Go modules that require authentication to access
  • Provide a standardized mechanism across all workflows to handle private dependencies when GOPRIVATE is configured
  • Allow the Go toolchain to authenticate with private GitHub repositories without requiring manual configuration in each workflow

Testing Performed

  • Verify workflows continue to function correctly with the new github-token parameter added
  • Test that private module authentication step is skipped when GOPRIVATE is not set (existing behavior unchanged)
  • Validate that git configuration is properly set when both token and GOPRIVATE are provided

Impact / Risk

  • Low Risk: Changes are backward compatible - the authentication step only runs conditionally when both token and GOPRIVATE are present
  • No Breaking Changes: Existing workflows without private modules continue to work unchanged
  • Security Consideration: Token is passed as environment variable and used only for git configuration, following GitHub's recommended authentication pattern

@mrz1836 mrz1836 self-assigned this Mar 4, 2026
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Mar 4, 2026
@github-actions github-actions bot added update General updates size/M Medium change (51–200 lines) labels Mar 4, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

Code Coverage Analysis

🟡 Overall Coverage: 85.0%
No Go files modified in this PR

Project coverage remains at 85.0% (3.3K/3.9K statements)

Changes: 16 config files

Coverage Metrics

Metric Value Grade Trend
Percentage 85.0% ⭐ B+ 📊
Statements 3.3K/3.9K ⭐ B+ No change
Quality Score 85/100 ⭐ B+ 📈

Coverage Breakdown

████████████░░░ 85.0%

Resources


Generated via go-coverage2026-03-04 20:23:09 UTC

github-actions bot added a commit that referenced this pull request Mar 4, 2026
🏷️ Generated with GoFortress
@mrz1836
Copy link
Owner Author

mrz1836 commented Mar 4, 2026

LGTM!

@mrz1836 mrz1836 merged commit 7a80056 into master Mar 4, 2026
44 checks passed
@github-actions github-actions bot deleted the chore/sync-files-mrz-tools-20260304-151742-6bf8447 branch March 4, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/M Medium change (51–200 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant