Skip to content

Enable jsdoc-format lint rule#14985

Merged
1 commit merged into
masterfrom
jsdoc-format
Apr 5, 2017
Merged

Enable jsdoc-format lint rule#14985
1 commit merged into
masterfrom
jsdoc-format

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Apr 3, 2017

We had a few different styles of jsdoc; this makes them all the same.

@ajafff
Copy link
Copy Markdown
Contributor

ajafff commented Apr 3, 2017

You diff contains ^M in some files. Is that intentional?

@ghost ghost force-pushed the jsdoc-format branch from 1c1b4ee to a728117 Compare April 3, 2017 21:23
@ghost
Copy link
Copy Markdown
Author

ghost commented Apr 3, 2017

Not intentional. The file was CRLF before and it's CRLF now, so I really don't understand why git is randomly showing the CR as added. I don't notice anything special about harnessLanguageService.ts either. They're all UTF-8 too.

@ghost ghost force-pushed the jsdoc-format branch from a728117 to 73cab09 Compare April 3, 2017 21:39
@ghost ghost merged commit 2b61d18 into master Apr 5, 2017
@ghost ghost deleted the jsdoc-format branch April 5, 2017 14:19
@microsoft microsoft locked and limited conversation to collaborators Jun 21, 2018
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants