-
-
Notifications
You must be signed in to change notification settings - Fork 5
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 github_release
alias for travis_release
task
#28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 the travis
is confusing!
Maybe it is more future-proof to rename the existing implementation and add a task with the old name for backward compatibility?
And maybe we can also have a release
task that also run github_release
for even more future-proof-proofness 😄 ?
I also thought more about a generic name. |
@ekohl I thought about it, but I wanted to avoid collisions and I wasn't sure which namespace is the best. Do you have a suggestion? |
e7b964e
to
c422af1
Compare
c422af1
to
4e165b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -33,6 +33,11 @@ | |||
g.push! | |||
end | |||
|
|||
desc 'Release via GitHub Actions (alias for travis_release)' | |||
task :release do | |||
Rake::Task['travis_release'].invoke |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I'd have preferred the other way around. Rename travis_release
to release
and add a travis_release
for backwards compatibility. Then travis_release
should also output a warning that it's deprecated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we did that in #32
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah thanks. I see it didn't add a warning. That can still be a good addition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No description provided.