Skip to content

[WIP] Add notify job to sync workflow in GitHub Actions#51

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/add-notify-job-to-sync
Closed

[WIP] Add notify job to sync workflow in GitHub Actions#51
Copilot wants to merge 1 commit intomainfrom
copilot/add-notify-job-to-sync

Conversation

Copy link

Copilot AI commented Feb 6, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

Context

A change was already implemented in the Geth generic repo that adds a Discord notification job to the sync workflow:
dappnode/DAppNodePackage-geth-generic@7552d5b

This same change needs to be applied to this repository.

Task

Add a new notify job at the END of the .github/workflows/sync.yml file.

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

Requirements

  1. Open the file .github/workflows/sync.yml
  2. Add the notify job at the END of the workflow file (after all existing jobs)
  3. Do NOT modify any other jobs
  4. Do NOT refactor workflow formatting
  5. Do NOT reorder existing jobs
  6. The notify job must depend on sync
  7. Ensure indentation remains valid YAML

Branch and Commit

  • Create branch: add-sync-notify-job
  • Commit message: ci: add discord notify job to sync workflow
  • Open PR to main
  • Do NOT merge the PR

This pull request was created from Copilot chat.


💡 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.

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