Skip to content

[Sync] Update project files from source repository (a86b40f)#256

Merged
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-bitcoin-schema-20260120-091600-a86b40f
Jan 20, 2026
Merged

[Sync] Update project files from source repository (a86b40f)#256
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-bitcoin-schema-20260120-091600-a86b40f

Conversation

@mrz1836
Copy link
Member

@mrz1836 mrz1836 commented Jan 20, 2026

What Changed

  • Updated MAGE_X_VERSION from v1.18.0 to v1.18.1 in .github/.env.base
  • Removed the entire .github/actions/test-statistics/action.yml composite action file (371 lines)
  • The removed action was marked as deprecated with a note that magex native CI mode now automatically produces .mage-x/ci-results.jsonl with test statistics

Why It Was Necessary

  • The custom test statistics collection action has been superseded by built-in functionality in magex CI mode
  • magex now automatically detects GitHub Actions and generates comprehensive test results including pass/fail counts, duration metrics, and failure details
  • Removing deprecated code reduces maintenance burden and eliminates redundant functionality that is now natively supported

Testing Performed

  • Verify that workflows using magex test:* commands continue to generate .mage-x/ci-results.jsonl output
  • Confirm validation workflows can successfully read test statistics from the ci-results.jsonl file
  • Ensure no workflows still reference the removed test-statistics composite action

Impact / Risk

  • Low Risk: The removed action was already deprecated and marked for removal; functionality is replaced by native magex features
  • No Breaking Changes: Workflows already using magex CI mode are unaffected as they rely on ci-results.jsonl output
  • Requires Migration: Any workflows still calling the test-statistics action directly will need to be updated to use magex native CI mode

@mrz1836 mrz1836 self-assigned this Jan 20, 2026
Copilot AI review requested due to automatic review settings January 20, 2026 14:16
@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 Jan 20, 2026
@github-actions github-actions bot added update General updates size/L Large change (201–500 lines) labels Jan 20, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR synchronizes project files from the source repository (mrz1836/go-broadcast) to remove deprecated test statistics collection infrastructure that has been superseded by native magex CI mode functionality.

Changes:

  • Updated MAGE_X_VERSION from v1.18.0 to v1.18.1
  • Removed deprecated test-statistics composite action (371 lines) that collected test metrics manually
  • Removed test-statistics action usage from fuzz testing workflow

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/.env.base Updated MAGE_X_VERSION to v1.18.1 for latest magex features
.github/actions/test-statistics/action.yml Removed entire deprecated action file that is superseded by magex native CI mode
.github/workflows/fortress-test-fuzz.yml Removed deprecated test-statistics step; workflow already uses .mage-x/ci-results-fuzz.jsonl output

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mrz1836 mrz1836 merged commit 0eef697 into master Jan 20, 2026
53 checks passed
@github-actions github-actions bot deleted the chore/sync-files-bitcoin-schema-20260120-091600-a86b40f branch January 20, 2026 14:22
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/L Large change (201–500 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants