Merge lp:~cjwatson/postgresfixture/ship-tests into lp:postgresfixture

Proposed by Colin Watson
Status: Merged
Merged at revision: 17
Proposed branch: lp:~cjwatson/postgresfixture/ship-tests
Merge into: lp:postgresfixture
Diff against target: 25 lines (+5/-3)
1 file modified
setup.py (+5/-3)
To merge this branch: bzr merge lp:~cjwatson/postgresfixture/ship-tests
Reviewer Review Type Date Requested Status
Gavin Panella Approve
William Grant (community) code Approve
Review via email: mp+371180@code.launchpad.net

Commit message

Include tests in sdist.

Description of the change

This would make it a bit nicer to package postgresfixture for (say) Debian and run its tests while building the package, which is something we generally like to do to make sure the distribution hasn't regressed tests relative to the last upstream release.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) :
review: Approve (code)
Revision history for this message
Gavin Panella (allenap) wrote :

LGTM. Sorry for taking so long; I missed the email from Launchpad. My fault.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'setup.py'
2--- setup.py 2017-11-28 07:29:24 +0000
3+++ setup.py 2019-08-11 22:18:45 +0000
4@@ -13,7 +13,10 @@
5
6 import codecs
7
8-from setuptools import setup
9+from setuptools import (
10+ find_packages,
11+ setup,
12+ )
13
14
15 with codecs.open("requirements.txt", "rb", encoding="utf-8") as fd:
16@@ -32,8 +35,7 @@
17 'Programming Language :: Python :: 3',
18 'Topic :: Software Development :: Libraries',
19 ],
20- packages={'postgresfixture'},
21- package_dir={'postgresfixture': 'postgresfixture'},
22+ packages=find_packages(),
23 install_requires=requirements,
24 tests_require=("testtools >= 0.9.14",),
25 test_suite="postgresfixture.tests",

Subscribers

People subscribed via source and target branches

to all changes: