Skip to content

Add another example and glob reference for lint.per-file-ignores - #28106

Merged
ntBre merged 9 commits into
astral-sh:mainfrom
chirizxc:patch-3
Aug 28, 2026
Merged

ntBre merged 9 commits into
astral-sh:mainfrom
chirizxc:patch-3

Conversation

@chirizxc

@chirizxc chirizxc commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

I came across this syntax while exploring the glob crate and found it quite useful. I think it would be worth adding a short example to the documentation.

Test Plan

Updated mimalloc version from 0.1.49 to 0.1.52 with v3 features.
Updated mimalloc feature from 'v3' to 'win_direct_tls'.
@astral-sh-bot

astral-sh-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me, I just had one suggestion for an additional improvement while we're here.

Comment thread crates/ruff_workspace/src/options.rs Resolved
@ntBre ntBre added the documentation Improvements or additions to public-facing documentation label Aug 27, 2026

@ntBre ntBre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

@ntBre ntBre changed the title docs: add one more example for docs (lint.per-file-ignores) Add another example and glob reference for lint.per-file-ignores Aug 28, 2026
@ntBre
ntBre merged commit 088b7b5 into astral-sh:main Aug 28, 2026
49 checks passed
@chirizxc
chirizxc deleted the patch-3 branch August 28, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to public-facing documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants