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

Update input_to_output method arguments to use dissimilar names #333

Merged
merged 1 commit into from Nov 9, 2020

Conversation

connorshea
Copy link
Contributor

This method using _ for both arguments was causing Sorbet to fail when generating type information (sorbet/sorbet#3561). This change doesn't modify any functionality or outward API in the gem itself. The argument names are based on the arguments for input_to_output in lib/bootsnap/compile_cache/yaml.rb.

This method using `_` for both arguments was causing Sorbet to fail when generating type information (sorbet/sorbet#3561). This change doesn't modify any functionality or outward API in the gem itself. The argument names are based on the arguments for input_to_output in `lib/bootsnap/compile_cache/yaml.rb`.
@ghost ghost added the cla-needed label Nov 7, 2020
@connorshea connorshea closed this Nov 7, 2020
@connorshea connorshea reopened this Nov 7, 2020
@connorshea
Copy link
Contributor Author

I'm not sure how to get the CLA job to restart but I have signed it

@ghost ghost removed the cla-needed label Nov 9, 2020
@rafaelfranca rafaelfranca merged commit a415dda into Shopify:master Nov 9, 2020
@casperisfine casperisfine temporarily deployed to rubygems November 10, 2020 09:17 Inactive
@connorshea connorshea deleted the patch-1 branch November 27, 2020 19:06
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.

None yet

3 participants