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

Fixes nose run test on test case #1225

Merged

Conversation

efremovd
Copy link
Contributor

@efremovd efremovd commented Sep 20, 2017

Hi,

First, thank you for your awesome work.

Fixes #597

I went through doc for nose and looks like test should be run as <file_name>:<class name> so I fixed nameToRun to have such syntax.

@efremovd
Copy link
Contributor Author

did they ever pass?

@DonJayamanne
Copy link
Owner

@efremovd
I've tested the changes & it looks great. Unfortunately I too made some changes to this file.
Please could you resolve the conflicts so I can merge this in.
Thanks.

@efremovd
Copy link
Contributor Author

@DonJayamanne will do later today. Thank you.

@efremovd
Copy link
Contributor Author

@DonJayamanne I resolved conflicts, please review. Thank you.

@DonJayamanne DonJayamanne merged commit bb61d26 into DonJayamanne:master Sep 22, 2017
DonJayamanne added a commit that referenced this pull request Apr 4, 2018
#1225)

* split compilation for faster compilation on windows
* 🔨 rename linting (else auto fix is not available) [skip ci]
* ✅ tests
* Fixes #1143
* Also updated debugger (npm) packages.
Goom11 pushed a commit to mostafaeweda/pythonVSCode that referenced this pull request Aug 30, 2018
DonJayamanne#1225)

* split compilation for faster compilation on windows
* 🔨 rename linting (else auto fix is not available) [skip ci]
* ✅ tests
* Fixes #1143
* Also updated debugger (npm) packages.
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