Skip to content

README.md: Adjust Compiler Explorer link wording#3180

Merged
StephanTLavavej merged 1 commit intomicrosoft:mainfrom
Alcaro:patch-1
Oct 27, 2022
Merged

README.md: Adjust Compiler Explorer link wording#3180
StephanTLavavej merged 1 commit intomicrosoft:mainfrom
Alcaro:patch-1

Conversation

@Alcaro
Copy link
Contributor

@Alcaro Alcaro commented Oct 26, 2022

I asked Matt Godbolt how he'd prefer this sentence, he said

I don't have a strong opinion, I'd probably just say Compiler Explorer if it were up to me though

@Alcaro Alcaro requested a review from a team as a code owner October 26, 2022 11:21
@StephanTLavavej StephanTLavavej added the documentation Related to documentation or comments label Oct 26, 2022
@StephanTLavavej StephanTLavavej self-assigned this Oct 26, 2022
@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 99e4c99 into microsoft:main Oct 27, 2022
@StephanTLavavej
Copy link
Member

Thanks for checking with Matt and updating the README - congratulations on your first microsoft/STL commit! ✅ 🎉 😺

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

Labels

documentation Related to documentation or comments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants