-
Notifications
You must be signed in to change notification settings - Fork 188
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
Two minor fixes #2662
Two minor fixes #2662
Conversation
Needs tests. |
For the 3rd issue, a simple repro is:
|
@chrisseaton Do you have local changes on this branch (if so please push them)? |
No I don't think we wrote anything else did we? There was the change to Paquito, but that was just to work around the |
OK, I'm happy to integrate these fixes and add tests then. |
All done now, I'll close this and we'll just need to merge those 3 PRs linked above. |
PullRequest: truffleruby/3366
…ecs (#2662) PullRequest: truffleruby/3367
All 3 fixes merged now. |
No description provided.