Skip to content
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

Updated dev dependency requirements #262

Merged
merged 1 commit into from
Aug 7, 2019
Merged

Updated dev dependency requirements #262

merged 1 commit into from
Aug 7, 2019

Conversation

geleems
Copy link

@geleems geleems commented Aug 7, 2019

  • pytest
    • current installation version is 3.2.2, which conflicts with pytest-cov-2.7.1 requirement. pytest-cov-2.7.1 requires pytest>=3.6
    • pytest-4.6.5 is latest compatible version for both Python 2 and 3.
  • nose
    • We are now using nose instead of pytest for auto test discovery in vscode python extension since there is an issue on pytest test discovery, which keeps throwing error message saying pytest is not installed while it is actually installed -- not resolved yet.
      • pytest is still being used for running unit tests in command prompt.
    • nose-1.3.7 is latest compatible version for both Python 2 and 3.
  • pylint
    • Very useful tool for detecting wrong space characters and link breakers in vscode.
      • Message pops up suggesting use of pylint when opening mssql-cli workspace with vscode.
    • pylint-1.9.5 is latest compatible version for both Python 2 and 3.

@geleems geleems requested a review from pensivebrian August 7, 2019 19:55
@geleems geleems self-assigned this Aug 7, 2019
@geleems geleems merged commit be187af into master Aug 7, 2019
@geleems geleems deleted the gelee/guide branch August 7, 2019 20:20
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