Skip to content

[Sync] Update project files from source repository (4ee089a)#6

Merged
mrz1836 merged 1 commit intomainfrom
chore/sync-files-bsv-blockchain-services-20260208-183340-4ee089a
Feb 8, 2026
Merged

[Sync] Update project files from source repository (4ee089a)#6
mrz1836 merged 1 commit intomainfrom
chore/sync-files-bsv-blockchain-services-20260208-183340-4ee089a

Conversation

@mrz1836
Copy link
Copy Markdown
Contributor

@mrz1836 mrz1836 commented Feb 8, 2026

What Changed

  • Removed the monolithic .github/.env.base file (550 lines) containing all GoFortress configuration defaults
  • Split configuration into modular .github/env/*.env files organized by concern:
    • 00-core.env - Go version and module configuration
    • 10-coverage.env - Coverage tool configuration
    • 10-mage-x.env - MAGE-X task runner configuration
    • 10-pre-commit.env - Pre-commit hook configuration
    • 10-security.env - Security scanning configuration
    • 20-redis.env - Redis service configuration
    • 20-workflows.env - GitHub workflow settings
  • Added .github/env/load-env.sh script to source environment files in correct order
  • Added .github/env/README.md documenting the new modular environment system
  • Updated .github/actions/load-env/action.yml to use the new load-env.sh script instead of direct .env.base loading
  • Updated all workflow files to reference the new modular environment system
  • Added .github/.yamlfmt for YAML formatting standards
  • Updated documentation in .github/docs/workflows.md and .github/docs/repository-features.md to reflect modular configuration approach
  • Updated tech conventions documentation for pre-commit and commit/branch standards

Why It Was Necessary

  • The 550-line monolithic .env.base file was difficult to navigate, maintain, and understand which settings belonged to which tools
  • Modular configuration allows teams to override only specific tool settings without duplicating the entire configuration file
  • Alphabetically-ordered loading (00-, 10-, 20- prefixes) provides predictable precedence and makes dependencies between configuration sections explicit

Testing Performed

  • Verified all 32 modified files show consistent references to the new .github/env/ directory structure
  • Confirmed load-env.sh script implements proper precedence (numbered files, then .env.custom overrides)
  • Validated that workflow files correctly invoke the updated load-env action
  • Checked documentation accurately describes the new modular environment system and migration path

Impact / Risk

  • Breaking Change: Projects relying on .env.base must migrate to the new modular structure or create .env.custom with their previous overrides
  • Low Risk: The load-env.sh script maintains backward compatibility by supporting .env.custom overrides
  • Configuration Clarity: Improved maintainability and discoverability of settings, reducing risk of misconfiguration
  • Migration Required: Teams must review their .env.custom files to ensure compatibility with the new modular structure

@mrz1836 mrz1836 self-assigned this Feb 8, 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 Feb 8, 2026
@github-actions github-actions bot added size/XL Very large change (>500 lines) update General updates labels Feb 8, 2026
@mrz1836 mrz1836 merged commit 176409e into main Feb 8, 2026
38 of 41 checks passed
@github-actions github-actions bot deleted the chore/sync-files-bsv-blockchain-services-20260208-183340-4ee089a branch February 8, 2026 23:45
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/XL Very large change (>500 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant