Skip to content

Add regression tests and close out already fixed fuzzball issues #10007

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

Merged
merged 1 commit into from
Oct 19, 2020

Conversation

griggt
Copy link
Contributor

@griggt griggt commented Oct 15, 2020

Add regression tests for the following open fuzzball issues that have already been fixed (either intentionally or incidentally):

#7745 is fixed by 9d158e0 in #9559
#9292 is fixed by 167db24 in #9845
#9330 is fixed by 9d158e0 in #9559
#9331 is fixed by 311e5ce in #9991
#9332 is fixed by 0bac80a in #9678

Note that although #9331 is "fixed" (it shows an error message at the appropriate location rather than crashes),
the error message is not particularly great, see #9331 (comment). Although, package objects are going away...

Closes #7745
Closes #9292
Closes #9330
Closes #9331 ?
Closes #9332

Copy link
Member

@dottybot dottybot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, and thank you for opening this PR! 🎉

All contributors have signed the CLA, thank you! ❤️

Have an awesome day! ☀️

Copy link
Contributor

@liufengyun liufengyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @griggt 🎉

@liufengyun liufengyun merged commit 8225502 into scala:master Oct 19, 2020
@griggt griggt deleted the close-fuzzballs branch October 19, 2020 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants