close
The Wayback Machine - https://web.archive.org/web/20201207075504/https://github.com/actions/create-release/pull/53
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

Default tag_name and release_name to be `github.ref` #53

Open
wants to merge 2 commits into
base: master
from

Conversation

@jasonkarns
Copy link

@jasonkarns jasonkarns commented Mar 27, 2020

For most workflows (indeed, even in the example for this action), this action will be scoped to only run for version tags. So for most workflows, it is highliy likely that the tag-name for the release ought to be the same as ${{ github.ref }}. In the spirit of convention over configuration, that would make a desirable default.

Similarly for the release_name. This actually results in the same behavior as when creating a release manually through the UI (the default release name is the same as the tag name, unless given explicitly).

(Pattern of setting defaults via expression directly in the manifest taken from actions/checkout.)

jasonkarns added 2 commits Mar 27, 2020
@jasonkarns jasonkarns changed the title Tag default Default tag_name and release_name to be `github.ref` Mar 27, 2020
@jasonkarns
Copy link
Author

@jasonkarns jasonkarns commented Mar 27, 2020

closes #26

@shivapoudel
Copy link

@shivapoudel shivapoudel commented May 5, 2020

@jasonkarns Additionally please remove the refs/tags portion of the string from body:

const body = core.getInput('body', { required: false }).replace('refs/tags/', '');

Currently, it is displaying like this:
image

@jasonkarns
Copy link
Author

@jasonkarns jasonkarns commented May 5, 2020

@shivapoudel I'm not against that, but doing so would be a more invasive change that would decrease the chance this PR gets merged. Perhaps open as a separate PR?

@shivapoudel
Copy link

@shivapoudel shivapoudel commented May 5, 2020

@jasonkarns Thanks for the suggestion, I have raised a PR #66 for this separately.

@jasonkarns
Copy link
Author

@jasonkarns jasonkarns commented Jul 13, 2020

bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.