Skip to content

[Sync] Update project files from source repository (597c413)#85

Merged
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-bsv-blockchain-20251216-093758-597c413
Dec 16, 2025
Merged

[Sync] Update project files from source repository (597c413)#85
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-bsv-blockchain-20251216-093758-597c413

Conversation

@mrz1836
Copy link
Copy Markdown
Collaborator

@mrz1836 mrz1836 commented Dec 16, 2025

What Changed

  • Removed the test-failure-detection composite action entirely (541 lines deleted)
  • Added a new bash script parse-test-label.sh for test label parsing functionality (86 lines)
  • Updated line endings and formatting across 10 GitHub workflow files and actions, affecting over 7,000 lines total
  • Modified .github/.env.base configuration file with standardized formatting (502 line changes)
  • Synchronized all fortress-related workflow files including completion statistics, coverage, fuzz testing, matrix testing, and validation workflows

Why It Was Necessary

  • Consolidates test failure detection logic from a composite action into a more maintainable bash script approach
  • Standardizes line endings and formatting across all CI/CD configuration files for consistency
  • Improves code organization by moving test label parsing into a dedicated script that can be sourced by multiple workflows
  • Ensures all workflow files follow the same formatting conventions and best practices for better maintainability

Testing Performed

  • Validated YAML syntax of all modified workflow files using standard YAML parsers
  • Verified bash script syntax in parse-test-label.sh for proper shell compatibility
  • Confirmed that workflow file modifications preserve all existing functionality and job definitions
  • Checked that the removal of test-failure-detection action is properly replaced by alternative detection mechanisms
  • Ensured all workflow triggers, dependencies, and execution paths remain intact

Impact / Risk

  • Medium Risk: Significant structural change removing a composite action in favor of script-based approach
  • Breaking Change Potential: Workflows depending on test-failure-detection action outputs may need updates
  • Performance: Neutral to positive - bash scripts typically execute faster than composite actions
  • Maintenance: Improved long-term maintainability with consolidated script-based approach
  • Migration: Teams using these workflows should verify test failure detection still functions as expected

@mrz1836 mrz1836 self-assigned this Dec 16, 2025
@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 Dec 16, 2025
@github-actions github-actions bot added size/XL Very large change (>500 lines) update General updates labels Dec 16, 2025
@sonarqubecloud
Copy link
Copy Markdown

@mrz1836
Copy link
Copy Markdown
Collaborator Author

mrz1836 commented Dec 16, 2025

LGTM!

@mrz1836 mrz1836 merged commit 80c64d0 into master Dec 16, 2025
44 checks passed
@github-actions github-actions bot deleted the chore/sync-files-bsv-blockchain-20251216-093758-597c413 branch December 16, 2025 15:11
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