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

Removing subprocess from Nikola's check #1591

Merged
merged 1 commit into from Jan 19, 2015
Merged

Removing subprocess from Nikola's check #1591

merged 1 commit into from Jan 19, 2015

Conversation

@felixfontein
Copy link
Contributor

felixfontein commented Jan 18, 2015

Implements part of things discussed in #1586. I mainly created a pull request so that we see what AppVeyor does with this.

… Nikola again.

Also avoiding unnecessary console output.
@Kwpolska Kwpolska added this to the v7.3.1 milestone Jan 19, 2015
@Kwpolska Kwpolska self-assigned this Jan 19, 2015
Kwpolska added a commit that referenced this pull request Jan 19, 2015
Removing subprocess from Nikola's check

close #1586 (setuptools' fault)
@Kwpolska Kwpolska merged commit 3777ba1 into master Jan 19, 2015
2 checks passed
2 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci The Travis CI build passed
Details
@Kwpolska Kwpolska deleted the check-nuke-subprocess branch Jan 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.