-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Backports 1.5.4 #39351
Backports 1.5.4 #39351
Changes from all commits
fd3af72
fd30588
65f39a7
8dbf798
ae8c3a0
4f70156
24a4637
d179f48
94af03e
b9ba10a
a4ac1f9
9fc38b8
7b05f42
8684c3d
1c3326f
cb75d49
c39f8c5
81c044a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
a5532dcc43089e5c3590903c647dd532 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
36bb45035c0a52a7c826b2b6574cd220c91d46f2a71b4b023514a0f6ad9132c072bdb09e83127c36c96ad03eff63d04d231c6aef716eeb51f13d2eb3d5625e54 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
3ebd2aaf5c2deedf89cb04f72e88f4e3 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
3198642bee9b06b48c99102f4f4a24d7d8bfc7691d890a3596ecb433ea75c1b0d4d9ff55161735bbfae071f2e570ef16f6b14ba350a822bd451324069eee19aa |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
2e5e802747e9101c4c70bd88d98f0aa7 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
a21fb98e33cd901ec1bfc79163687f6cb6dac02406fb3ebd33c6d8b63309b66cbe850c9da608bbc1a61a967a4309dafc308431d40c6ba8beab837e345f526f5e |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
0661f12762e2f2bbc90bce17d90fdbcf |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
a0f40ca80578d3d9e6b94a7f11d292ecd52de90374645c8fa8607e28b9f421709fbbba941ee3e7c9fba88abfe87daf6d265dbe8fcaf0e2faf1b83d8b5ab8a9a7 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
5adc7739a9c5d4568870e2852c85485b |
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.
@Sacha0 - i also did this backport, here (oops, we collided. I'm happy to drop that now that you've done it 😊):
backports-release-1.5...NHDaly:nhd-backports-release-1.5-manual
And i wasn't sure whether to pull over the
isa Type
change as well. Technically that was added in dcc0696, as part of (#36596), so i wasn't sure whether to backport it as well. Supposedly it's just a code cleanup, but i'm not sure if it holds in isolation from the rest of the commit.I think the closest transformation keeping in spirit of the change we're backporting from (#37645) would be just to drop the
Module
part, since that's all they did in the original PR? Like this, perhaps?:PROBABLY fine either way, but since i see you made a different decision than I made when attempting the manual bisect, I wanted to just bring this up! :)
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.
To see the diff standalone, it's the change i made:
cc09911#diff-94ffb7fc1a419bb8439ae0ecbb4a3b5460d193e53c96bb97b50e283fdbc16a85L932-R932
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.
I wondered the same, but err'd to adding the
isa Type
guard as== Type{Union{}}
cannot be true unless the guard is true, and if the latter is true then the former must be true as well, so this transformation should be functionally identical modulo some additional safety with the guard (against== Type{Union{}}
throwing something for non-Type
s0
s)? Either way is fine by me though :).