Skip to content

Fix some Kestrel test indenting #43164

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
Merged
merged 1 commit into from
Aug 9, 2022
Merged

Fix some Kestrel test indenting #43164

merged 1 commit into from
Aug 9, 2022

Conversation

JamesNK
Copy link
Member

@JamesNK JamesNK commented Aug 9, 2022 •
edited
Loading

Messed up indenting triggers my OCD. Half an hour of manual fixing 😬

Background: #38681 (comment)

adityamandaleeka
adityamandaleeka approved these changes Aug 9, 2022
View reviewed changes
Copy link
Member

@adityamandaleeka adityamandaleeka left a comment

Choose a reason for hiding this comment

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

I see you sneaking in those vars 😆

@Tratcher Tratcher merged commit e3404f4 into main Aug 9, 2022
@Tratcher Tratcher deleted the jamesnk/kestrel-indenting branch August 9, 2022 21:44
@ghost ghost added this to the 7.0-rc1 milestone Aug 9, 2022
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adityamandaleeka adityamandaleeka adityamandaleeka approved these changes

@Tratcher Tratcher Awaiting requested review from Tratcher

@halter73 halter73 Awaiting requested review from halter73 halter73 is a code owner

@BrennanConroy BrennanConroy Awaiting requested review from BrennanConroy BrennanConroy is a code owner

Assignees
No one assigned
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Projects
None yet
Milestone
7.0-rc1
Development

Successfully merging this pull request may close these issues.
4 participants
@JamesNK @adityamandaleeka @Tratcher @amcasey