Skip to content

Respect custom help builder during parse error reporting - #2211

Merged
adamsitnik merged 2 commits into
dotnet:mainfrom
adamsitnik:respectCustomHelpBuilderInParseAction
Jun 6, 2023
Merged

Respect custom help builder during parse error reporting#2211
adamsitnik merged 2 commits into
dotnet:mainfrom
adamsitnik:respectCustomHelpBuilderInParseAction

Conversation

@adamsitnik

@adamsitnik adamsitnik commented Jun 6, 2023

Copy link
Copy Markdown
Member

It may be the last bug for dotnet/sdk#29131 (at leas I hope so)

@adamsitnik
adamsitnik requested a review from jonsequitur June 6, 2023 14:50
@adamsitnik
adamsitnik enabled auto-merge (squash) June 6, 2023 14:52
@jonsequitur

Copy link
Copy Markdown
Contributor

This conflicts with my changes in #2205, where I've removed the EnableParseErrorReporting property.

@adamsitnik

Copy link
Copy Markdown
Member Author

This conflicts with my changes in #2205, where I've removed the EnableParseErrorReporting property.

I am aware of that. This fixes solves the last bug that stops us from updating the SDK:

https://dev.azure.com/dnceng-public/public/_build/results?buildId=297080&view=ms.vss-test-web.build-test-results-tab

image

it would be really great to merge to be able to merge dotnet/sdk#29131

@adamsitnik
adamsitnik merged commit bf313ca into dotnet:main Jun 6, 2023
@adamsitnik
adamsitnik deleted the respectCustomHelpBuilderInParseAction branch June 6, 2023 18:28
@adamsitnik adamsitnik mentioned this pull request Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants