Skip to content

Remove always-on WarningWhenInliningMethodImplNoInlineMarkedFunction feature flag - #20310

Merged
T-Gro merged 2 commits into
mainfrom
fix/issue-20191
Aug 24, 2026
Merged

Remove always-on WarningWhenInliningMethodImplNoInlineMarkedFunction feature flag#20310
T-Gro merged 2 commits into
mainfrom
fix/issue-20191

Conversation

@T-Gro

Copy link
Copy Markdown
Member

Fixes#20191

WarningWhenInliningMethodImplNoInlineMarkedFunction has been on for every selectable language version since F# 8.0, so its flag was dead configuration. The warning for let inline combined with [<MethodImpl(MethodImplOptions.NoInlining)>] is now unconditional, and --disableLanguageFeature:WarningWhenInliningMethodImplNoInlineMarkedFunction is no longer a recognised feature name.

@github-actions

github-actionsBot commented Aug 21, 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-GroT-Gro added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Aug 21, 2026
@github-actionsgithub-actionsBot added the AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed label Aug 21, 2026
… feature
Remove the WarningWhenInliningMethodImplNoInlineMarkedFunction language
feature flag and make the FS3151 warning for 'let inline' bindings marked
[<MethodImpl(MethodImplOptions.NoInlining)>] unconditional.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@T-Gro
T-Gro enabled auto-merge (squash) August 24, 2026 13:45
@T-Gro
T-Gro requested a review from abonieAugust 24, 2026 13:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Remove always-on language feature flag: WarningWhenInliningMethodImplNoInlineMarkedFunction

2 participants

@T-Gro@abonie