- Notifications
You must be signed in to change notification settings - Fork131
adapt to scala/scala#9292#306
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
SethTisue commentedOct 29, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
first commit was a blind stab, second commit was after I looked at it a bit closer. we can squash before merge, if this ends up being merged |
som-snytt commentedOct 30, 2020
🩸 🔪 🙈 |
som-snytt commentedOct 30, 2020
I would like to make the same offer as posco on twitter ("if you would like me to leave suggestions and comments"), except with puns and emoji. |
SethTisue commentedOct 30, 2020
two puns please, and a side order of emoji, hold the pickle |
SethTisue commentedNov 3, 2020
squashed. regardless of what happens toscala/scala#9292, I think this is code improvement and we should just merge it. |
No description provided.