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

Optimize Options Source Gen when no need to run #93427

Merged

Conversation

tarekgh
Copy link
Member

@tarekgh tarekgh commented Oct 12, 2023

This should contribute to improvements in the results of speedometer and Visual Studio (VS) tests, which have exhibited regressions. This contribute a little to #93313 too.

@ghost
Copy link

ghost commented Oct 12, 2023

Tagging subscribers to this area: @dotnet/area-extensions-options
See info in area-owners.md if you want to be subscribed.

Issue Details

This should contribute to improvements in the results of speedometer and Visual Studio (VS) tests, which have exhibited regressions.

Author: tarekgh
Assignees: tarekgh
Labels:

area-Extensions-Options

Milestone: -

@tarekgh tarekgh added the source-generator Indicates an issue with a source generator feature label Oct 12, 2023
@tarekgh tarekgh added this to the 8.0.0 milestone Oct 12, 2023
@tarekgh
Copy link
Member Author

tarekgh commented Oct 13, 2023

failures are tracked by #92423 which are unrelated. I am not sure why the build analysis didn't detect it.

@tarekgh tarekgh merged commit bcec497 into dotnet:main Oct 13, 2023
104 of 109 checks passed
@tarekgh tarekgh deleted the OptimizeOptionsSourceGenWhenNoNeedToRun branch October 13, 2023 16:20
@tarekgh
Copy link
Member Author

tarekgh commented Oct 13, 2023

/backport to release/8.0

@github-actions
Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/6510949878

@arkalyanms
Copy link
Member

@tarekgh is this already in VS main? would you point me to the insertion PR if yes? We would like to coordinate Roslyn side paired changes post that.

cc: @CyrusNajmabadi

@ghost ghost locked as resolved and limited conversation to collaborators Nov 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Extensions-Options source-generator Indicates an issue with a source generator feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants