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
lazygit 0.38.1 #129939
lazygit 0.38.1 #129939
Conversation
db8ef27
to
75a15ec
Compare
The last PR had an issue with the test so I fixed that up |
Formula/lazygit.rb
Outdated
assert_match "foo", (testpath/"git-rebase-todo").read | ||
ENV["LAZYGIT_DAEMON_KIND"] = "1" # exit immediately | ||
ENV["LAZYGIT_DAEMON_INSTRUCTION"] = "{}" | ||
ok = system "#{bin}/lazygit" |
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.
Wouldn't this just print the help text?
Formula/lazygit.rb
Outdated
# assert that the command exits with exit code 0 | ||
assert_equal true, ok |
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.
System already fails if it does
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'm surprised if that's true because the test fails upon comparing true to nil. If system raised an exception upon failure, I wouldn't expect it to reach the next line.
EDIT: actually it's just occurred to me that this system
method may be homebrew-specific, and that it returns nil upon success. That makes sense.
75a15ec
to
ed73776
Compare
system "#{bin}/lazygit", "git-rebase-todo" | ||
assert_match "foo", (testpath/"git-rebase-todo").read | ||
assert_match "pick 401a0c3", (testpath/"git-rebase-todo").read |
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 original issue was that we changed the interface here so I'm now just updating to reflect that
I'm gonna close this and reopen a PR for v0.38.2 :) |
action-homebrew-bump-formula
Created with
brew bump-formula-pr
.