Skip to content

[Sync] Update project files from source repository (824fa4e)#113

Merged
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-mrz-tools-20260213-162558-824fa4e
Feb 13, 2026
Merged

[Sync] Update project files from source repository (824fa4e)#113
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-mrz-tools-20260213-162558-824fa4e

Conversation

@mrz1836
Copy link
Owner

@mrz1836 mrz1836 commented Feb 13, 2026

What Changed

  • Updated documentation comments to reflect configuration now being loaded from modular .github/env/*.env files instead of .env.base and .env.custom files (updated in header comment and environment variables section)
  • Added conditional execution to the load-env job to only run for fork PRs using if: ${{ github.event.pull_request.head.repo.full_name != github.repository }}
  • Added comment clarifying that same-repo PRs are handled by pull-request-management.yml workflow
  • Added visual separator line (════════════════════════════════════════════════════════════════════════════════) in the header
  • Minor whitespace adjustments in security scanner justification comment block for improved alignment

Why It Was Necessary

  • The conditional execution prevents the fork workflow from running unnecessarily on same-repository PRs, improving workflow efficiency and reducing redundant job execution
  • Documentation updates reflect the migration to a modular environment configuration system, ensuring developers have accurate information about where configuration is managed
  • The visual separator improves readability and organization of the extensive security documentation in the workflow header

Testing Performed

  • Workflow syntax validation through GitHub Actions schema
  • Verification that the conditional logic correctly identifies fork vs same-repo PRs using github.event.pull_request.head.repo.full_name comparison
  • Review of documentation changes for accuracy and consistency across all references to configuration file locations

Impact / Risk

  • Low Risk: Changes are primarily documentation updates and a conditional guard that prevents unnecessary execution
  • No Breaking Changes: The workflow will continue to function for fork PRs as intended; same-repo PRs will simply skip this job (as they should)
  • Improved Efficiency: Prevents duplicate workflow runs and clarifies the separation of concerns between fork and same-repo PR workflows

@mrz1836 mrz1836 self-assigned this Feb 13, 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 13, 2026
@github-actions github-actions bot added size/S Small change (11–50 lines) update General updates labels Feb 13, 2026
@github-actions
Copy link
Contributor

Code Coverage Analysis

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

Project coverage remains at 85.2% (3.2K/3.7K statements)

Changes: 1 config file

Coverage Metrics

Metric Value Grade Trend
Percentage 85.2% ⭐ B+ 📊
Statements 3.2K/3.7K ⭐ B+ No change
Quality Score 85.2/100 ⭐ B+ 📈

Coverage Breakdown

████████████░░░ 85.2%

Resources


Generated via go-coverage2026-02-13 21:32:00 UTC

github-actions bot added a commit that referenced this pull request Feb 13, 2026
🏷️ Generated with GoFortress
@mrz1836 mrz1836 merged commit 34bc79d into master Feb 13, 2026
44 checks passed
@github-actions github-actions bot deleted the chore/sync-files-mrz-tools-20260213-162558-824fa4e branch February 13, 2026 22:07
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/S Small change (11–50 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant