Skip to content

Make use of shared workflows#950

Open
acroca wants to merge 3 commits intodapr:mainfrom
acroca:shared-workflows
Open

Make use of shared workflows#950
acroca wants to merge 3 commits intodapr:mainfrom
acroca:shared-workflows

Conversation

@acroca
Copy link
Copy Markdown
Member

@acroca acroca commented Mar 10, 2026

@acroca acroca requested review from a team as code owners March 10, 2026 13:20
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.06%. Comparing base (bffb749) to head (727a214).
⚠️ Report is 103 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #950      +/-   ##
==========================================
- Coverage   86.63%   81.06%   -5.58%     
==========================================
  Files          84      137      +53     
  Lines        4473    13184    +8711     
==========================================
+ Hits         3875    10687    +6812     
- Misses        598     2497    +1899     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@acroca acroca marked this pull request as draft March 11, 2026 09:56
@acroca
Copy link
Copy Markdown
Member Author

acroca commented Mar 11, 2026

Will bring back the deleted pieces, as discussed here: dapr/go-sdk#815

acroca added 2 commits March 16, 2026 17:41
Signed-off-by: Albert Callarisa <albert@diagrid.io>
Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca acroca force-pushed the shared-workflows branch from 5e0713d to c943265 Compare March 16, 2026 16:41
@acroca acroca marked this pull request as ready for review March 16, 2026 16:42
Copy link
Copy Markdown
Contributor

@sicoyle sicoyle left a comment

Choose a reason for hiding this comment

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

one comment pls and then this is g2g 🚀 thank you!!

uses: dapr/.github/.github/actions/setup-dapr-runtime@main
with:
go-version: ${{ env.GOVER }}
commit: ${{ github.event.inputs.daprdapr_commit }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why is this daprdapr_commit?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it's the name of the input, I didn't change it in this commit, I guess it's because it references the dapr/dapr repo...

Signed-off-by: Albert Callarisa <albert@diagrid.io>
Copy link
Copy Markdown
Contributor

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 updates the repository’s GitHub Actions configuration to use shared workflows/actions from dapr/.github, reducing duplicated workflow logic maintained in this repo.

Changes:

  • Replaced in-repo automerge and stale-pruning jobs with reusable workflows from dapr/.github.
  • Replaced Dapr CLI/runtime setup logic in validate_examples with shared setup actions.
  • Removed the local .github/scripts/automerge.py script that is no longer referenced.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
.github/workflows/validate_examples.yaml Uses shared actions to install/setup Dapr CLI and runtime during example validation runs.
.github/workflows/dapr-bot-schedule.yml Switches automerge + stale pruning to shared reusable workflows.
.github/workflows/backport.yaml Switches backport automation to a shared reusable workflow.
.github/scripts/automerge.py Deletes the legacy automerge script now superseded by shared workflows.

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

Comment thread .github/workflows/validate_examples.yaml
Comment thread .github/workflows/validate_examples.yaml
Comment thread .github/workflows/dapr-bot-schedule.yml
Comment thread .github/workflows/dapr-bot-schedule.yml
Comment thread .github/workflows/backport.yaml
Comment thread .github/workflows/backport.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants