-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Demote i686-pc-windows-gnu to Tier 2 #137869
base: master
Are you sure you want to change the base?
Demote i686-pc-windows-gnu to Tier 2 #137869
Conversation
r? Kobzol you can take a look at the CI stuff already |
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.
Infra changes look good, left one comment. Good riddance, 32-bit legacy mess!
@@ -528,31 +527,6 @@ auto: | |||
# came from the mingw-w64 SourceForge download site. Unfortunately | |||
# SourceForge is notoriously flaky, so we mirror it on our own infrastructure. | |||
|
|||
# i686-mingw is split into three jobs to run tests in parallel. |
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.
❤️
cb71349
to
5e115aa
Compare
In accordance with RFC 3771. I also added a stub doc page for the target and renamed the windows-gnullvm page for consistency.
5e115aa
to
77677fe
Compare
These commits modify compiler targets. |
r? workingjubilee I want to give you the honor of the bors r plus |
# Set of tests that should represent half of the time of the test suite. | ||
# Used to split tests across multiple CI runners. |
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.
Nit: I believe this comment is still accurate (see below for ci-mingw-x
and ci-mingw-bootstrap
. cc @marcoieni since I could be mistaken.
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.
yes, those two lines are still accurate for ci-mingw-x
and ci-mingw-bootstrap
👍
Can you bring them back?
Thanks for noticing!
In accordance with RFC 3771. FCP has been completed.
I also added a stub doc page for the target and renamed the windows-gnullvm page for consistency.