Skip to content

Conversation

@scrabsha
Copy link
Contributor

@scrabsha scrabsha commented Nov 26, 2025

Part of #131229.

r? @jdonszelmann

I think it's worth running the Windows test suite before merging that (I don't have the rights for this).

@rustbot
Copy link
Collaborator

rustbot commented Nov 26, 2025

Some changes occurred in compiler/rustc_hir/src/attrs

cc @jdonszelmann

Some changes occurred in compiler/rustc_passes/src/check_attr.rs

cc @jdonszelmann

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann

@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 26, 2025
@rustbot

This comment has been minimized.

@jdonszelmann
Copy link
Contributor

uh, I think this should be the right job:
@bors try jobs=x86_64-msvc

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Nov 26, 2025
Port the `#![windows_subsystem]` attribute to the new attribute system

try-job: x86_64-msvc
@rust-bors

This comment was marked as off-topic.

@rust-log-analyzer

This comment was marked as off-topic.

@jieyouxu
Copy link
Member

Close, missing a -1.
@bors try jobs=x86_64-msvc-1

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Nov 27, 2025
Port the `#![windows_subsystem]` attribute to the new attribute system

try-job: x86_64-msvc-1
@jdonszelmann
Copy link
Contributor

oops, thanks jieyou!

@rust-bors
Copy link

rust-bors bot commented Nov 27, 2025

☀️ Try build successful (CI)
Build commit: a16778f (a16778f2dc05e6f49d23e40e50cb67acf80b0f3e, parent: a4175159da185130408f71d7aa7e9bff0d455c24)

@jdonszelmann
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Nov 27, 2025

📌 Commit 2ab2090 has been approved by jdonszelmann

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 27, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Nov 27, 2025
…r=jdonszelmann

Port the `#![windows_subsystem]` attribute to the new attribute system

Part of rust-lang#131229.

I think it's worth running the Windows test suite before merging that (I don't have the rights for this).
bors added a commit that referenced this pull request Nov 27, 2025
Rollup of 8 pull requests

Successful merges:

 - #147071 (constify from_fn, try_from_fn, try_map, map)
 - #148930 (tweak editor configs)
 - #149320 (-Znext-solver: normalize expected function input types when fudging)
 - #149363 (Port the `#![windows_subsystem]` attribute to the new attribute system)
 - #149378 (make run-make tests use 2024 edition by default)
 - #149381 (Add `impl TrustedLen` on `BTree{Map,Set}` iterators)
 - #149388 (remove session+blob decoder construction)
 - #149390 (`rust-analyzer` subtree update)

r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-attributes Area: Attributes (`#[…]`, `#![…]`) S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants