-
Notifications
You must be signed in to change notification settings - Fork 54
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
reana-dev: update git commands to use gh CLI #723
Conversation
d3d1d13
to
fbb2270
Compare
PS: a rebase is needed! 😉 |
fbb2270
to
d115e90
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #723 +/- ##
==========================================
+ Coverage 18.93% 18.94% +0.01%
==========================================
Files 26 26
Lines 2271 2285 +14
==========================================
+ Hits 430 433 +3
- Misses 1841 1852 +11
|
b2b8144
to
9183d21
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.
Works well, just one small comment left! This PR also needs to be rebased
9183d21
to
4b02a06
Compare
reana/reana_dev/git.py
Outdated
component, | ||
) | ||
else: | ||
display_message( |
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.
display_message
cannot be used when --automatic
is not provided, as the line prefix (i.e. date and component) printed by display_message
interferes with eval when running eval "$(reana-dev git-fork -c ALL -b firefox)"
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.
Right - my bad! Reverted it to click.echo
.
4b02a06
to
ebf1ac0
Compare
Closes #483.