Skip to content

ci: add discord notify job to sync workflow#84

Draft
Copilot wants to merge 1 commit intomainfrom
copilot/add-notify-job-to-workflow
Draft

ci: add discord notify job to sync workflow#84
Copilot wants to merge 1 commit intomainfrom
copilot/add-notify-job-to-workflow

Conversation

Copy link

Copilot AI commented Feb 6, 2026

Adds Discord notification to the sync workflow to alert on job completion status.

Changes

  • Added notify job to .github/workflows/sync.yml
  • Job triggers after sync completes (success or failure, but not skipped)
  • Calls reusable workflow dappnode/staker-test-util/.github/workflows/notify-discord.yml@main
  • Passes sync result and test type parameters
  • Inherits secrets for Discord webhook authentication

Implementation

notify:
  needs: sync
  if: always() && needs.sync.result != 'skipped'
  uses: dappnode/staker-test-util/.github/workflows/notify-discord.yml@main
  with:
    result: ${{ needs.sync.result }}
    test_type: "sync"
  secrets: inherit

This change aligns with the notification pattern already implemented in DAppNodePackage-geth-generic.

Original prompt

You need to apply the same GitHub Actions workflow change across multiple repositories.

Context

A change was already implemented in the Geth generic repo:

7552d5b

The change adds a new notify job at the end of .github/workflows/sync.yml.

The job to add is:

notify:
  needs: sync
  if: always() && needs.sync.result != 'skipped'
  uses: dappnode/staker-test-util/.github/workflows/notify-discord.yml@main
  with:
    result: ${{ needs.sync.result }}
    test_type: "sync"
  secrets: inherit

Target repositories

Execution clients:

Consensus clients:

Requirements

For EACH repository:

  1. Clone the repository.

  2. Create a new branch:

    add-sync-notify-job
    
  3. Open the file:

    .github/workflows/sync.yml
    
  4. Add the notify job at the END of the workflow file.

  5. Do not modify any other jobs.

  6. Commit with message:

    ci: add discord notify job to sync workflow
    
  7. Push the branch.

  8. Open a Pull Request to main.

  9. Do NOT merge the PR.

Important constraints

  • Assume .github/workflows/sync.yml already exists in every repo.
  • Do not refactor workflow formatting.
  • Do not reorder jobs.
  • Do not modify the existing sync job.
  • The notify job must depend on sync.
  • Ensure indentation remains valid YAML.

Expected output

Provide:

  • A list of created PR URLs
  • Confirmation that all repositories were updated
  • Any repositories that failed (if applicable)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add notify job to GitHub Actions workflow ci: add discord notify job to sync workflow Feb 6, 2026
Copilot AI requested a review from Marketen February 6, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants