Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove fallbacks #74041

Merged
merged 4 commits into from
Jul 9, 2024
Merged

Remove fallbacks #74041

merged 4 commits into from
Jul 9, 2024

Conversation

tmat
Copy link
Member

@tmat tmat commented Jun 18, 2024

Remove options fallbacks.

Fixes
#60777
#60786

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Jun 18, 2024
@tmat tmat force-pushed the RemoveFallbacks1 branch 2 times, most recently from 538d7a6 to fc441c2 Compare June 18, 2024 20:41
@tmat tmat force-pushed the RemoveFallbacks1 branch 3 times, most recently from df2991f to fd80d9d Compare June 20, 2024 19:21
@tmat tmat changed the title Remove fallbacks1 Remove fallbacks - 1 of n Jun 20, 2024
@tmat tmat force-pushed the RemoveFallbacks1 branch from 6301858 to 33bc1f8 Compare June 28, 2024 21:43
@CyrusNajmabadi
Copy link
Member

image

@CyrusNajmabadi
Copy link
Member

Who do you think you are? Me?

@tmat tmat force-pushed the RemoveFallbacks1 branch 2 times, most recently from c586cc1 to f523bf8 Compare July 2, 2024 15:38
@tmat tmat force-pushed the RemoveFallbacks1 branch from f523bf8 to 5a029f2 Compare July 9, 2024 01:13
@tmat tmat force-pushed the RemoveFallbacks1 branch from 5a029f2 to 8bc83af Compare July 9, 2024 15:08
@tmat tmat changed the title Remove fallbacks - 1 of n Remove fallbacks Jul 9, 2024
@tmat tmat marked this pull request as ready for review July 9, 2024 16:04
@tmat tmat requested a review from a team as a code owner July 9, 2024 16:04
};

public static CleanCodeGenerationOptions GetCleanCodeGenerationOptions(this IOptionsReader options, LanguageServices languageServices, bool? allowImportsInHiddenRegions, CleanCodeGenerationOptions? fallbackOptions)
public static CleanCodeGenerationOptions GetCleanCodeGenerationOptions(this IOptionsReader options, LanguageServices languageServices, bool? allowImportsInHiddenRegions = null)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would prefer non-optional parameters.

@tmat tmat enabled auto-merge (squash) July 9, 2024 17:36
@tmat tmat merged commit 6ceee07 into dotnet:main Jul 9, 2024
25 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants