Skip to content

Support triggable references (branches, tags). Ignore triggable references on GitHub action #546

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

Merged
merged 9 commits into from
Jun 7, 2020

Conversation

jetersen
Copy link
Member

@jetersen jetersen commented Jun 1, 2020

While exploring this issue I found many issues that this PR will greatly improve the experience for.

  • Allow GitHub action to always trigger, let the GitHub workflow decide the trigger ;)
  • GraphQL supports receiving the fully qualified reference, so let us just pass that.
  • some dead flatten code, actually handled by Joi.
  • Support ref/tags when running on GitHub app

cc @toolmantim @TimonVS

There should be minimal impact as seen in the test changes we support both branches and references definition.

@arminc this should fully unlock any GitHub workflow you desire.

Note to self: Release notes should make clear branches is renamed to references

fixes #550
fixes #545
fixes #515
fixes #542
fixes #527
fixes #520
fixes #404
fixes #203
fixes #141

jetersen added 4 commits June 1, 2020 03:47
This should allow GitHub Action and GitHub app which
allows us to work with any reference
See https://developer.github.com/v4/object/repository/#ref
This change is actually backwards compatible thanks to Joi
@jetersen
Copy link
Member Author

jetersen commented Jun 3, 2020

@darkdreamingdan would you mind reviewing this PR 🙏

@SebastianThiebaud
Copy link

This is great. These changes would allow this action to work in a git flow context. Thanks for submitting this PR. I will patiently wait for a merge :)

@jetersen jetersen merged commit 5e3cbad into release-drafter:master Jun 7, 2020
@jetersen jetersen deleted the fix/references branch June 7, 2020 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
2 participants