Skip to content

Conversation

@mattmccutchen-cci
Copy link
Member

No description provided.

Copy link
Member

@mwhicks1 mwhicks1 left a comment

Choose a reason for hiding this comment

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

All of this looks great to me. Thanks, Matt!

@mattmccutchen-cci mattmccutchen-cci changed the title Documentation update pass 2021-02 3C documentation update pass 2021-02 Feb 26, 2021
@mattmccutchen-cci mattmccutchen-cci marked this pull request as ready for review February 26, 2021 22:00
@mattmccutchen-cci
Copy link
Member Author

This is all the documentation I want to add for now. Mike, do you want to finish reviewing?

Copy link
Member

@mwhicks1 mwhicks1 left a comment

Choose a reason for hiding this comment

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

Looks fantastic! Just a few changes requested.

Copy link
Member Author

@mattmccutchen-cci mattmccutchen-cci left a comment

Choose a reason for hiding this comment

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

Thanks for the thoughtful review.

Copy link
Member

@mwhicks1 mwhicks1 left a comment

Choose a reason for hiding this comment

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

I'd still like to see that one para deleted (explanation added) but the rest looks good. Thanks!

@mattmccutchen-cci mattmccutchen-cci dismissed mwhicks1’s stale review February 28, 2021 20:33

Mike told me elsewhere that this is approved.

@mattmccutchen-cci mattmccutchen-cci merged commit ff245c1 into main Feb 28, 2021
@mattmccutchen-cci mattmccutchen-cci deleted the documentation-202102 branch February 28, 2021 20:35
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.

Verify fix for bogus reports of line ending changes in a clean git checkout

3 participants