Skip to content

Minor documentation updates (Matt's version) #628

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

Merged
merged 2 commits into from
Jun 22, 2021
Merged

Conversation

mattmccutchen-cci
Copy link
Member

This is my counterproposal to #607, attempting to address the same problems that #607 sought out to address. Ideally, I'd like to get it into our imminent omnibus PR to Microsoft.

@mwhicks1 Can you provide an additional opinion on whether you think Microsoft will be OK with this change to the root readme (or maybe even ask them now)? Or I guess we can just include it and revert it later if they ask during review of the omnibus PR.

@aaronjeline
Copy link
Collaborator

Yeah that makes things easier to find. Thanks!

@mattmccutchen-cci
Copy link
Member Author

Mike said in the meeting that he thinks the root readme change should be OK, so I'll go ahead and merge this.

@mattmccutchen-cci mattmccutchen-cci removed the request for review from mwhicks1 June 22, 2021 18:28
@mattmccutchen-cci mattmccutchen-cci merged commit da7b8f5 into main Jun 22, 2021
@mattmccutchen-cci mattmccutchen-cci deleted the docupdates-matt branch June 22, 2021 18:29
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.

2 participants