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

Implement IManagedHotReloadAgent2 and IManagedHotReloadAgent4 for ProjectHotReloadSession #9555

Merged
merged 27 commits into from
Nov 14, 2024

Conversation

LittleLittleCloud
Copy link
Contributor

@LittleLittleCloud LittleLittleCloud commented Oct 11, 2024

This PR implements IManagedHotReloadAgent4 for ProjectHotReloadSession. So that debugger can query it for the full path to its csproj file and use it for restarting the corresponding project.

@drewnoakes
Copy link
Member

why webtool's ProjectHotReloadSession implements IManagedHotReloadAgent2

You can use "blame" on the WebTools source to see it was added in 2022 with this PR https://devdiv.visualstudio.com/DevDiv/_git/WebTools/pullrequest/371614 in response to bug https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1453640/

I don't know more than that though.

@LittleLittleCloud LittleLittleCloud changed the title Implement IManagedHotReloadAgent2 for ProjectHotReloadSession Implement IManagedHotReloadAgent2 and IManagedHotReloadAgent4 for ProjectHotReloadSession Oct 15, 2024
nuget.config Outdated Show resolved Hide resolved
@drewnoakes drewnoakes self-requested a review October 17, 2024 14:15
@drewnoakes drewnoakes dismissed their stale review October 17, 2024 14:16

Code has changed quite a bit since then. Will review again once the PR is published.

@LittleLittleCloud LittleLittleCloud marked this pull request as ready for review October 21, 2024 21:14
@LittleLittleCloud LittleLittleCloud requested a review from a team as a code owner October 21, 2024 21:14
@LittleLittleCloud
Copy link
Contributor Author

/azp run

Copy link

Commenter does not have sufficient privileges for PR 9555 in repo dotnet/project-system

@LittleLittleCloud
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@adamint
Copy link
Member

adamint commented Oct 22, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@LittleLittleCloud
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@drewnoakes drewnoakes left a comment

Choose a reason for hiding this comment

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

Looking good. A few comments below.

LittleLittleCloud and others added 3 commits November 13, 2024 17:54
…tem/VS/HotReload/ProjectHotReloadSession.cs

Co-authored-by: Drew Noakes <[email protected]>
…tem/VS/HotReload/IProjectHotReloadSessionCallback.cs

Co-authored-by: Drew Noakes <[email protected]>
Copy link
Member

@drewnoakes drewnoakes left a comment

Choose a reason for hiding this comment

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

Looks great.

@LittleLittleCloud LittleLittleCloud merged commit c1539d2 into dotnet:main Nov 14, 2024
5 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 17.12 milestone Nov 14, 2024
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.

4 participants