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.

Sign up for GitHub

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

Jump to bottom

Re-order Debugging Start-up to be like v1. Refactor MeadowConnectionManager so it can be shared amongst extensions #590

Closed
wants to merge 12 commits into from
Closed

Re-order Debugging Start-up to be like v1. Refactor MeadowConnectionManager so it can be shared amongst extensions #590

wants to merge 12 commits into from

Conversation

adrianstevens
Copy link
Contributor

No description provided.

@CartBlanche CartBlanche self-assigned this Aug 21, 2024
@CartBlanche CartBlanche requested a review from ctacke August 21, 2024 08:37
@CartBlanche CartBlanche removed their assignment Aug 21, 2024
@CartBlanche CartBlanche changed the title Dominique test debugging Re-order Debugging Start-up to be like v1. Refactor MeadowConnectionManager so it can be shared amongst extensions Aug 21, 2024
@CartBlanche CartBlanche self-assigned this Aug 21, 2024
@CartBlanche CartBlanche removed the request for review from ctacke August 22, 2024 16:39
@CartBlanche CartBlanche force-pushed the dominique-TestDebugging branch from d69e8d5 to db724f8 Compare September 4, 2024 10:29
@CartBlanche CartBlanche force-pushed the dominique-TestDebugging branch 2 times, most recently from 97ea8d7 to 0f184ea Compare September 27, 2024 01:55
CartBlanche and others added 11 commits October 15, 2024 12:25
@CartBlanche
Move StartDebugging so the start-up order is more like v1.
fb94748
@CartBlanche
Make GetConnectionForRoute awaitable, add locks and remove Thread.Sle…
fc431a4 
…ep and goto
@adrianstevens @CartBlanche
Restored tabs to spaces (autoformat on save)
f277060
@CartBlanche
If we use ConcurrentQueue, we don't need AutoReset, as it should be t…
cf0b84f 
…hread/task safe.
@CartBlanche
BlockingCollection is already thread safe, so shouldn't need AutoReset
9da4f7b
@CartBlanche
Refactor to Move reponse parsing to it's own method. Avoid Thread.Sle…
7de2280 
…ep to stop conext switching.
@CartBlanche
Commented out, unused/referenced code for now. Moved MeadowForMeadowA…
d873ac1 
…ttach to where it is used most, cleaning up ListernProc.
@CartBlanche
Revert "Commented out, unused/referenced code for now. Moved MeadowFo…
189440f 
…rMeadowAttach to where it is used most, cleaning up ListernProc."

This reverts commit e7aab15.
@CartBlanche
Add missing Exception catch, that allows debugging to continue in VS4…
4f07574 
…M and VSCode.
@CartBlanche
Use specificed timeout to get debugging working with VSCode and VSMac.
daaade8
@CartBlanche
Bump to upload-artifact V4 (v2 deprecated) and version *.60 for testing.
6754ae7
@CartBlanche CartBlanche force-pushed the dominique-TestDebugging branch from 787736a to 6754ae7 Compare October 15, 2024 01:34
@adrianstevens
Copy link
Contributor Author

This is out of sync now with Alexis' latest changes - closing PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ctacke ctacke ctacke left review comments

@CartBlanche CartBlanche CartBlanche left review comments

Assignees

@CartBlanche CartBlanche

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adrianstevens @CartBlanche @ctacke