-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Move rest of the warn tests from neg to warn (batch 3) #19244
Conversation
dd50793
to
4d111b9
Compare
7b3b400
to
69927a4
Compare
4d111b9
to
5bec325
Compare
69927a4
to
6d759fb
Compare
5bec325
to
d7fdaf3
Compare
6d759fb
to
78adb39
Compare
d7fdaf3
to
9f3ef59
Compare
78adb39
to
2994d53
Compare
9f3ef59
to
b84a68b
Compare
fb8e104
to
f21b0f3
Compare
1e740e2
to
69d1de3
Compare
c96a43c
to
2b68e23
Compare
2b68e23
to
90cb205
Compare
@@ -1,13 +0,0 @@ | |||
//> using options -Xfatal-warnings | |||
|
|||
class Test { |
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.
It's in tests/warn already, was just left behind
93bb0c2
to
c54633f
Compare
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.
Otherwise LGTM
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 one should not be changed yet
c54633f
to
039744e
Compare
039744e
to
bfe0c70
Compare
The last batch of tests, move tests other than tests/init/neg and tests/neg to warn.
PR 4/5 (merge consecutively, per Nicolas' suggestion
Split up version of #18829