Skip to content

Remove passing ILTrim tests from expected failures list#127288

Merged
jkotas merged 2 commits intomainfrom
copilot/update-iltrim-expected-failures
Apr 22, 2026
Merged

Remove passing ILTrim tests from expected failures list#127288
jkotas merged 2 commits intomainfrom
copilot/update-iltrim-expected-failures

Conversation

@jkotas
Copy link
Copy Markdown
Member

@jkotas jkotas commented Apr 22, 2026

No description provided.

Copilot AI and others added 2 commits April 22, 2026 05:30
Agent-Logs-Url: https://github.com/dotnet/runtime/sessions/a4a27125-931d-4421-ae6d-089cb082bdd9

Co-authored-by: MichalStrehovsky <13110571+MichalStrehovsky@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 22, 2026 16:29
@jkotas jkotas merged commit 639aaa7 into main Apr 22, 2026
123 of 143 checks passed
@jkotas jkotas deleted the copilot/update-iltrim-expected-failures branch April 22, 2026 16:29
@jkotas
Copy link
Copy Markdown
Member Author

jkotas commented Apr 22, 2026

CI run in #127263

@jkotas
Copy link
Copy Markdown
Member Author

jkotas commented Apr 22, 2026

cc @MichalStrehovsky

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the ILTrim test infrastructure by removing tests that now pass from the “expected failures” allowlist, keeping the CoreCLR ILTrim test suite’s failure expectations accurate.

Changes:

  • Removed FeatureSettings.FeatureDescriptors from the expected failures list.
  • Removed several LinkXml.* tests from the expected failures list now that they pass.

@MichalStrehovsky
Copy link
Copy Markdown
Member

cc @MichalStrehovsky

Thank you!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants