Skip to content

Comments

[syntax-errors]: future-feature-not-defined (F407)#20554

Merged
ntBre merged 2 commits intoastral-sh:mainfrom
11happy:F407
Sep 25, 2025
Merged

[syntax-errors]: future-feature-not-defined (F407)#20554
ntBre merged 2 commits intoastral-sh:mainfrom
11happy:F407

Conversation

@11happy
Copy link
Contributor

@11happy 11happy commented Sep 24, 2025

Summary

This PR implements https://docs.astral.sh/ruff/rules/future-feature-not-defined/ (F407) as a semantic syntax error.

Test Plan

I have written inline tests as directed in #17412

@11happy 11happy changed the title [synatx-errors]: future-feature-not-defined (F406) [synatx-errors]: future-feature-not-defined (F407) Sep 24, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 24, 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.

@11happy 11happy changed the title [synatx-errors]: future-feature-not-defined (F407) [syntax-errors]: future-feature-not-defined (F407) Sep 24, 2025
Copy link
Contributor

@ntBre ntBre 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! This looks great overall, just a few small suggestions

@ntBre ntBre added the internal An internal refactor or improvement label Sep 24, 2025
Copy link
Contributor

@ntBre ntBre 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 cfc64d1 into astral-sh:main Sep 25, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants