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
@ntBre ntBre added the documentation Improvements or additions to 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 documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants