Skip to content

Fix typo in Stack Trace Explorer text#60853

Merged
ryzngard merged 2 commits intodotnet:mainfrom
ryzngard:issues/ste_typo
Apr 21, 2022
Merged

Fix typo in Stack Trace Explorer text#60853
ryzngard merged 2 commits intodotnet:mainfrom
ryzngard:issues/ste_typo

Conversation

@ryzngard
Copy link
Copy Markdown
Contributor

Modify the text when a user needs to paste a stack trace to fix a typo and remove extra words.

@ryzngard ryzngard requested a review from a team as a code owner April 20, 2022 00:15
@ghost ghost added the Area-IDE label Apr 20, 2022
Copy link
Copy Markdown
Contributor

@CyrusNajmabadi CyrusNajmabadi left a comment

Choose a reason for hiding this comment

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

Do we need the trailing period?

@JoeRobich
Copy link
Copy Markdown
Member

/azp run roslyn-CI

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ryzngard
Copy link
Copy Markdown
Contributor Author

Do we need the trailing period?

I think in this instance it makes more sense. The text shows up in almost a paragraph format where a stack would show otherwise. I'm open to either though

@ryzngard ryzngard merged commit ca0faf7 into dotnet:main Apr 21, 2022
@ryzngard ryzngard deleted the issues/ste_typo branch April 21, 2022 17:14
@ghost ghost added this to the Next milestone Apr 21, 2022
@dibarbet dibarbet modified the milestones: Next, 17.3.P1 Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants