Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dispatch actions task to ping tools on tag #6755

Merged
merged 1 commit into from
Jun 22, 2024

Conversation

sciencewhiz
Copy link
Contributor

Allows automation of tagging of tools on allwpilib tag.
Adapted from robotpy

@sciencewhiz sciencewhiz requested a review from a team as a code owner June 19, 2024 01:57
@sciencewhiz
Copy link
Contributor Author

Here's an example of this working:
allwpilib: https://github.com/sciencewhiz/allwpilib-dispatch/actions/runs/9558211081/job/26347498473
SmartDashboard: https://github.com/sciencewhiz/SmartDashboard-dispatch/actions/runs/9558587336/job/26347501942

Requires a secret TOOLS_REPO_ACCESS_TOKEN PAT with read and write access to contents

Allows automation of tagging of tools on allwpilib tag
Adapted from robotpy
needs: [combine]
runs-on: ubuntu-22.04
steps:
- uses: peter-evans/repository-dispatch@v3
Copy link
Member

Choose a reason for hiding this comment

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

Any reason to use repo dispatch rather than workflow dispatch?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Oh when it was first added to robotpy, Github hadn't invented workflow_dispatch yet.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you have a recommendation for an action to trigger the workflow_dispatch? During a quick look through the actions marketplace, I didn't see any that seemed well maintained.

@sciencewhiz
Copy link
Contributor Author

The corresponding PR to tag Smartdashboard is here: wpilibsuite/SmartDashboard#192

@PeterJohnson PeterJohnson merged commit f5df6f8 into wpilibsuite:main Jun 22, 2024
32 checks passed
@sciencewhiz sciencewhiz deleted the PingTools branch June 22, 2024 17:41
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.

4 participants