Skip to content

Fix missing combine call for lint.typing-extensions setting#17823

Merged
ntBre merged 2 commits intomainfrom
brent/fix-missing-combine
May 3, 2025
Merged

Fix missing combine call for lint.typing-extensions setting#17823
ntBre merged 2 commits intomainfrom
brent/fix-missing-combine

Conversation

@ntBre
Copy link
Contributor

@ntBre ntBre commented May 3, 2025

Summary

Fixes #17821.

Test Plan

New CLI test. This might be overkill for such a simple fix, but it made me feel better to add a test.

@ntBre ntBre added bug Something isn't working configuration Related to settings and configuration labels May 3, 2025
@ntBre
Copy link
Contributor Author

ntBre commented May 3, 2025

Re-running the ecosystem check, which failed to be acquired by a runner. I'll merge as soon as it passes

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre merged commit fe4051b into main May 3, 2025
55 of 56 checks passed
@ntBre ntBre deleted the brent/fix-missing-combine branch May 3, 2025 22:19
Glyphack pushed a commit to Glyphack/ruff that referenced this pull request May 6, 2025
…al-sh#17823)

## Summary

Fixes astral-sh#17821.

## Test Plan

New CLI test. This might be overkill for such a simple fix, but it made
me feel better to add a test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working configuration Related to settings and configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lint.typing-extensions not respected

2 participants