-
Notifications
You must be signed in to change notification settings - Fork 35
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
[multimod] Remove tag push, print instead. #202
Conversation
Codecov ReportBase: 58.26% // Head: 58.17% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #202 +/- ##
==========================================
- Coverage 58.26% 58.17% -0.10%
==========================================
Files 44 44
Lines 1953 1927 -26
==========================================
- Hits 1138 1121 -17
+ Misses 685 680 -5
+ Partials 130 126 -4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@bogdandrutu maybe you want to review this? |
This PR removes the
--push
flag onmultimod tag
. As shown in issue #177push
required additional authentication to be set up since it did not natively read git credentials from the environment. Multiple options were discussed in #177. This PR implements the last suggestion which prints all tags separated by a newline. This--print-tags
option can then be worked into existing repositories release process.The
push
code has been removed from multimod. I believe theos.exec
option is still possible but I have not had time to address it. This option provides a quick win while also removing bugged code. I suggest we reopen #157 which can be assigned and investigated in the future.Fixes #177