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

reana-dev: update git commands to use gh CLI #723

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

giuseppe-steduto
Copy link
Member

Closes #483.

reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
reana/reana_dev/git.py Outdated Show resolved Hide resolved
@mdonadoni
Copy link
Member

PS: a rebase is needed! 😉

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #723 (ebf1ac0) into master (d4db226) will increase coverage by 0.01%.
The diff coverage is 21.73%.

Additional details and impacted files

Impacted file tree graph

@@            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     
Files Coverage Δ
reana/reana_dev/git.py 29.25% <21.73%> (-0.25%) ⬇️

@giuseppe-steduto giuseppe-steduto force-pushed the gh-tool-reana-dev branch 2 times, most recently from b2b8144 to 9183d21 Compare September 19, 2023 09:40
@giuseppe-steduto giuseppe-steduto marked this pull request as ready for review September 19, 2023 09:49
Copy link
Member

@mdonadoni mdonadoni left a 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

reana/reana_dev/git.py Outdated Show resolved Hide resolved
component,
)
else:
display_message(
Copy link
Member

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)"

Copy link
Member Author

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.

@mdonadoni mdonadoni merged commit ebf1ac0 into reanahub:master Oct 20, 2023
14 checks passed
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.

RFC {hub, ghi} -> gh
2 participants