Skip to content

Add a UI regression test. - #162665

Open
kulinsky wants to merge 1 commit into
rust-lang:mainfrom
kulinsky:test-link-section-foreign-static
Open

Add a UI regression test.#162665
kulinsky wants to merge 1 commit into
rust-lang:mainfrom
kulinsky:test-link-section-foreign-static

Conversation

@kulinsky

@kulinsky kulinsky commented Sep 11, 2026

Copy link
Copy Markdown

Added a UI test for #136220. The .stderr file contains the expected output; the test verifies that a warning appears when link_section is applied to a foreign static variable. The diagnostic issue has already been fixed; my commit confirms it with a test. The tests ./x test tests/ui/attributes/positions and ./x test tidy passed. LLM was used to navigate the repository.

A `link_section` attribute is applied to a foreign static. The UI test checks for a warning.
@rustbot rustbot added 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 Sep 11, 2026
@rustbot

rustbot commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the pull request, and welcome! The Rust Project has assigned @petrochenkov (or someone else) to review your changes, you should hear from them (or someone else) within the next two weeks.

Please see the contribution instructions and our LLM policy for more information.

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 76 candidates
  • Random selection from 20 candidates

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants