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

fix locale-sensitive comparison of UnmatchedArgument error strings #29746

Merged
merged 7 commits into from
Jan 12, 2023

Conversation

baronfel
Copy link
Member

@baronfel baronfel commented Jan 4, 2023

Forward-port of #29559 to the 7.0.2xx release branch so that we don't miss the merge window.

baronfel and others added 6 commits January 4, 2023 15:35
Co-authored-by: Amaury Levé <amaury.leve@gmail.com>
…f format string and check for ordered presence of those in the formatted message.
Adds a helper method for setting culture to the test framework, driven from a shared reference to the hard-coded UI environment variable.
@baronfel baronfel requested a review from a team as a code owner January 4, 2023 21:37
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@baronfel baronfel added this to the 7.0.2xx milestone Jan 4, 2023
@baronfel baronfel added Area-CLI cli-ux Issues and PRs that deal with the UX of the CLI (exit codes, log output, verbs/options, and so on) labels Jan 4, 2023
@baronfel baronfel removed the request for review from a team January 4, 2023 21:39
Copy link
Member

@dsplaisted dsplaisted left a comment

Choose a reason for hiding this comment

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

Looks good, same feedback as on the other version of this PR.

Co-authored-by: Daniel Plaisted <dsplaisted@gmail.com>
@baronfel
Copy link
Member Author

baronfel commented Jan 4, 2023

Applied your suggested change on that other PR and cherry-picked it here as well 👍

@baronfel
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@baronfel
Copy link
Member Author

Template engine leg is failing due to the moduleinitializer issue. @dotnet/templating-engine-maintainers how should I resolve this? did the moduleinitializer fixes get into 7.0.2xx?

@baronfel baronfel merged commit ca0e775 into dotnet:release/7.0.2xx Jan 12, 2023
@baronfel baronfel deleted the backport-29559-to-7.0.2xx branch January 12, 2023 01:38
@JanKrivanek
Copy link
Member

@baronfel Yes, fixed for 7.0.2xx (#29803)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-CLI cli-ux Issues and PRs that deal with the UX of the CLI (exit codes, log output, verbs/options, and so on)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants