Skip to content

Exclude broken symlinks from meson-python ecosystem check#17993

Merged
ntBre merged 3 commits intomainfrom
brent/remove-meson-python
May 9, 2025
Merged

Exclude broken symlinks from meson-python ecosystem check#17993
ntBre merged 3 commits intomainfrom
brent/remove-meson-python

Conversation

@ntBre
Copy link
Copy Markdown
Contributor

@ntBre ntBre commented May 9, 2025

Summary

This should resolve the formatter ecosystem errors we've been seeing lately. mesonbuild/meson-python#728 added the links, which I think are intentionally broken for testing purposes.

Test Plan

Ecosystem check on this PR

Summary
--

This should resolve the formatter ecosystem errors we've been seeing lately.
mesonbuild/meson-python#728 added the links, which I
think are intentionally broken for testing purposes.

Test Plan
--

Ecosystem check on this PR
@ntBre ntBre added the testing Related to testing Ruff itself label May 9, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 9, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@ntBre
Copy link
Copy Markdown
Contributor Author

ntBre commented May 9, 2025

This might be tricky to fix, but I guess you can't actually use {} in an --exclude glob since the comma causes clap to split the fields.

@ntBre ntBre requested a review from MichaReiser May 9, 2025 19:35
@ntBre ntBre marked this pull request as ready for review May 9, 2025 19:35
Copy link
Copy Markdown
Member

@MichaReiser MichaReiser left a comment

Choose a reason for hiding this comment

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

Thank you

@ntBre ntBre merged commit 40fd52d into main May 9, 2025
34 checks passed
@ntBre ntBre deleted the brent/remove-meson-python branch May 9, 2025 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Related to testing Ruff itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants