Skip to content

Conversation

@karrtikr
Copy link

@karrtikr karrtikr commented Aug 7, 2019

For #6117

@karrtikr karrtikr added the no-changelog No news entry required label Aug 7, 2019
@karrtikr karrtikr changed the title Add unit tests for client/linters/pylint.ts WIP - Added unit tests for client/linters/pylint.ts Aug 7, 2019
@karrtikr karrtikr changed the title WIP - Added unit tests for client/linters/pylint.ts Added unit tests for client/linters/pylint.ts Aug 7, 2019
Copy link

@kimadeline kimadeline left a comment

Choose a reason for hiding this comment

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

Mostly some tidbits here and there (nothing too serious)

@ericsnowcurrently ericsnowcurrently removed their request for review August 13, 2019 16:57
Kartik Raj and others added 9 commits August 13, 2019 10:19
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-Authored-By: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
@karrtikr karrtikr merged commit e5ed70b into microsoft:master Aug 14, 2019
@karrtikr karrtikr deleted the pylint branch August 14, 2019 17:46
@lock lock bot locked as resolved and limited conversation to collaborators Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants