Skip to content

Added articles about additional tooling and overall checklist#22213

Merged
CamSoper merged 8 commits intodotnet:masterfrom
DavidCBerry13:azure-tools
Jan 7, 2021
Merged

Added articles about additional tooling and overall checklist#22213
CamSoper merged 8 commits intodotnet:masterfrom
DavidCBerry13:azure-tools

Conversation

@DavidCBerry13
Copy link
Copy Markdown
Contributor

@DavidCBerry13 DavidCBerry13 commented Jan 4, 2021

@DavidCBerry13 DavidCBerry13 marked this pull request as ready for review January 5, 2021 05:22
Copy link
Copy Markdown
Member

@IEvangelist IEvangelist left a comment

Choose a reason for hiding this comment

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

A few minor nits...

DavidCBerry13 and others added 3 commits January 5, 2021 10:48
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
Copy link
Copy Markdown
Contributor

@CamSoper CamSoper left a comment

Choose a reason for hiding this comment

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

Couple of small things. Other than that LGTM :shipit:

DavidCBerry13 and others added 2 commits January 7, 2021 14:10
Co-authored-by: Cam Soper <cam.soper@microsoft.com>
Co-authored-by: Cam Soper <cam.soper@microsoft.com>
@CamSoper CamSoper merged commit 93a70ed into dotnet:master Jan 7, 2021
@gewarren
Copy link
Copy Markdown
Contributor

gewarren commented Jan 7, 2021

@CamSoper or @DavidCBerry13 Can you fix the warnings this PR introduced?

image

@CamSoper
Copy link
Copy Markdown
Contributor

CamSoper commented Jan 7, 2021

@gewarren dang it, I should have caught that. My bad.

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