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

Fix default archive commands for PRs #10

Merged
merged 4 commits into from
Jun 5, 2024
Merged

Conversation

emcfarlane
Copy link
Collaborator

@emcfarlane emcfarlane commented Jun 4, 2024

Fix invalid use of github.ref_name which points to main and not the pull request branch on delete events. We now correctly set the label as github.event.ref_name which points to the branch.

Copy link

github-actions bot commented Jun 4, 2024

The latest Buf updates on your PR.

NameStatus
build✅ passed
lint⏩ skipped
format❌ failed
breaking✅ passed

@emcfarlane emcfarlane assigned nicksnyder and unassigned nicksnyder Jun 4, 2024
@emcfarlane emcfarlane marked this pull request as draft June 4, 2024 16:46
@emcfarlane emcfarlane changed the title Disable archive on main Fix default archive commands for PRs Jun 4, 2024
@emcfarlane emcfarlane marked this pull request as ready for review June 4, 2024 17:14
@@ -198,4 +198,6 @@ jobs:
breaking: false
push: false
archive: true
# On push github.ref_name, on delete github.event.ref_name.
archive_labels: ${{ github.ref_name }}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is for testing, we tests a push then archive to roundtrip the label.

action.yml Outdated Show resolved Hide resolved
@emcfarlane emcfarlane merged commit d52db49 into main Jun 5, 2024
27 checks passed
@emcfarlane emcfarlane deleted the ed/fixArchiveOnMain branch June 5, 2024 14:34
@emcfarlane emcfarlane mentioned this pull request Jun 5, 2024
emcfarlane added a commit that referenced this pull request Jun 5, 2024
Whats changed:
- Fix default archive label in #10 
- Drop support for `.bufversion` files in #8 

Removed support for `.bufversion` files is a breaking change but is not
currently used by known implementations.
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