Skip to content

Remove always-on language feature flag PreferStringGetPinnableReference - #20370

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

Remove always-on language feature flag PreferStringGetPinnableReference#20370
T-Gro wants to merge 1 commit into
mainfrom
fix/issue-20186

Conversation

@T-Gro

@T-Gro T-Gro commented Aug 26, 2026

Copy link
Copy Markdown
Member

Fixes #20186

PreferStringGetPinnableReference shipped in F# 8.0 and is on for every selectable language version, so its flag was dead configuration. Removed the flag and collapsed the guarded TcAndBuildFixedExpr path to its always-on GetPinnableReference behavior.

…ce (#20186)

The PreferStringGetPinnableReference feature shipped in F# 8.0 and is on for every selectable language version (minimum langversion is 8.0), so its flag is dead configuration. Remove the LanguageFeature DU case, features-map entry, GetFeatureString arm, FSComp.txt resource string, and collapse the single guarded path in TcAndBuildFixedExpr to the always-on GetPinnableReference call. Regenerate xlf files.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@T-Gro
T-Gro requested a review from a team as a code owner August 26, 2026 15:39
@T-Gro T-Gro added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Aug 26, 2026
@github-actions github-actions Bot added the AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed label Aug 26, 2026
@T-Gro T-Gro added NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes and removed NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes labels Aug 27, 2026
@github-actions

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 enabled auto-merge (squash) August 27, 2026 09:13
@T-Gro
T-Gro requested a review from abonie August 27, 2026 09:13
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: PreferStringGetPinnableReference

1 participant