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

Add missing tests_require requirement (unittest2) #104

Merged
merged 1 commit into from Feb 27, 2015
Merged

Add missing tests_require requirement (unittest2) #104

merged 1 commit into from Feb 27, 2015

Conversation

koobs
Copy link
Contributor

@koobs koobs commented Jan 27, 2015

unittest2 is listed in test-requirements.txt, but is missing from tests_require in setup.py. It may be worth reading the contests of test_requirements.txt to populate tests_require, but I'd leave that design decision to you :) cc @dstufft

unittest2 is listed in test-requirements.txt, but is missing from tests_require in setup.py. It may be worth reading the contests of test_requirements.txt to populate tests_require, but I'd leave that design decision to you :) cc @dstufft
@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.39% when pulling 6bface4 on koobs:patch-1 into 51db425 on samgiles:master.

uqs pushed a commit to freebsd/freebsd-ports that referenced this pull request Jan 27, 2015
- Update to 0.6.1
- Chase upstream repository move [1]
- Add TESTS option
- Tweak existing OPTIONS descriptions
- Explicitly set TEST_DEPENDS
- Add LICENSE_FILE
- Update regression-test: target invocation
- Patch setup.py to include unittest2 requirement [2]
- pkg-descr: Update WWW: URL

[2] samgiles/slumber#104

PR:		197119 [1]
Reported by:	antoine [1]


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@377972 35697150-7ecd-e111-bb59-0022644237b5
koobs added a commit to freebsd/freebsd-ports that referenced this pull request Jan 27, 2015
- Update to 0.6.1
- Chase upstream repository move [1]
- Add TESTS option
- Tweak existing OPTIONS descriptions
- Explicitly set TEST_DEPENDS
- Add LICENSE_FILE
- Update regression-test: target invocation
- Patch setup.py to include unittest2 requirement [2]
- pkg-descr: Update WWW: URL

[2] samgiles/slumber#104

PR:		197119 [1]
Reported by:	antoine [1]
@samgiles
Copy link
Owner

Hi, thanks for this. I'll take a look at this when I have my laptop. I'm
the current maintainer of slumber BTW ;).

Sam

On Tue, 27 Jan 2015 02:58 Coveralls notifications@github.com wrote:

[image: Coverage Status] https://coveralls.io/builds/1810801

Coverage remained the same at 80.39% when pulling 6bface4
6bface4
on koobs:patch-1
into 51db425
51db425
on samgiles:master
.


Reply to this email directly or view it on GitHub
#104 (comment).

@koobs
Copy link
Contributor Author

koobs commented Jan 29, 2015

My pleasure Sam :)

@samgiles samgiles merged commit 6bface4 into samgiles:master Feb 27, 2015
svmhdvn pushed a commit to svmhdvn/freebsd-ports that referenced this pull request Jan 10, 2024
- Update to 0.6.1
- Chase upstream repository move [1]
- Add TESTS option
- Tweak existing OPTIONS descriptions
- Explicitly set TEST_DEPENDS
- Add LICENSE_FILE
- Update regression-test: target invocation
- Patch setup.py to include unittest2 requirement [2]
- pkg-descr: Update WWW: URL

[2] samgiles/slumber#104

PR:		197119 [1]
Reported by:	antoine [1]
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.

None yet

3 participants