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
utils/bottles: fix typechecking error #15381
Conversation
def tag(symbol = nil) | ||
return Tag.from_symbol(symbol) if symbol.present? | ||
return Tag.from_symbol(symbol.to_sym) if symbol.present? |
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.
IMO this should be fixed a level higher. The parameter is literally called "symbol" so should be a symbol.
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 thought that too (in part because I thought that might be able to fix it in one place instead of two), but I can't follow the call stack from the Sorbet error message. Happy to take a suggestion or replacement PR with a nicer fix.
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.
Ah, nvm, I got it.
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.
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
This is a better replacement of Homebrew#15381.
This is a better replacement of Homebrew#15381.
This is a better replacement of Homebrew#15381.
This is a better replacement of Homebrew#15381.
https://github.com/Homebrew/homebrew-core/actions/runs/4915420737/jobs/8777887066?pr=130438#step:5:27