Uh oh!
There was an error while loading. Please reload this page.
Simplify logic for IncludePlatformAttributes - #51575
Conversation
ghost
commented
Apr 20, 2021
Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer Issue DetailsAddressing some post-merge feedback on #51450 from @safern: #51450 (comment). This simplifies a difficult to read condition around when we should include the internal platform support attributes.
|
ghost
commented
Apr 20, 2021
Hello @jeffhandley! Because this pull request has the Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 10 minutes, a condition that will be fulfilled in about 5 minutes. No worries though, I will be back when the time is right! 😉 p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me ( |
jeffhandley
commented
Apr 20, 2021
The failing test is #51588. |
Addressing some post-merge feedback on #51450 from @safern: #51450 (comment).
This simplifies a difficult to read condition around when we should include the internal platform support attributes.