Skip to content

Remove always-on DotlessFloat32Literal language feature flag - #20441

Open
T-Gro wants to merge 1 commit into
mainfrom
fix/issue-20145
Open

Remove always-on DotlessFloat32Literal language feature flag#20441
T-Gro wants to merge 1 commit into
mainfrom
fix/issue-20145

Conversation

@T-Gro

@T-Gro T-Gro commented Sep 3, 2026

Copy link
Copy Markdown
Member

Fixes #20145

DotlessFloat32Literal shipped in F# 5.0 and is enabled for every selectable --langversion (minimum accepted is 8.0), so its feature flag can never be turned off. Removed the dead configuration and collapsed the guarded lexer rule to its always-on behaviour. --disableLanguageFeature:DotlessFloat32Literal is no longer a recognised feature name.

The DotlessFloat32Literal feature shipped in F# 5.0. Since the minimum
accepted --langversion is now 8.0, the flag is permanently enabled and is
dead configuration. Remove the LanguageFeature case, its features-map
entry, and GetFeatureString arm, and collapse the gated lexer rule into a
single always-on ieee32 rule by folding the dotless-integer case into the
ieee32 regex. Regenerated xlf localization files.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

⚠️ Release notes required, but author opted out

Warning

Author opted out of release notes, check is disabled for this pull request.
cc @dotnet/fsharp-team-msft

@T-Gro T-Gro added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Sep 3, 2026
@github-actions github-actions Bot added the AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed label Sep 3, 2026
@T-Gro
T-Gro requested a review from abonie September 4, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

Remove always-on language feature flag: DotlessFloat32Literal

1 participant