switch {pr,issue} view default#665
Conversation
mislav
left a comment
There was a problem hiding this comment.
Change looks good!
If we wanted to keep backwards compatibility with existing usages of issue/pr view -p, we could potentially keep the flag as supported, mark it as hidden from the docs, and have it be a no-op. However, since we're in beta, I have no issue with breaking backwards compatibility here. 👍
|
@mislav I disagree with this change and it did break existing workflows. More than that though, this should not have come out as a patch release, breaking existing functionality (even if you're in beta) should at least be a minor point release. |
|
@vs-jawad I'm sorry for the trouble. I agree that this should have not been a patch release due to this change. We're going to try to bump the minor version number for changes like these in the future. However, please note that this software is in beta and will be subject to many more changes until it can be considered "stable". Thank you. |
Summary
closes #555
Switches issue, pr view to print to the terminal by default and take
--web/-wto open issue/prin browser.