Skip to content

/ typed_ast

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

Jump to bottom

Distribute the tests #94

Merged
merged 1 commit into from Feb 9, 2019
Merged

Distribute the tests #94

merged 1 commit into from Feb 9, 2019
+3 −1

Conversation

@mr-c
Copy link
Contributor

@mr-c mr-c commented Feb 9, 2019

Patch from the Debian package for typed_ast, we use this to run the tests at build time and post-install. With this PR typed_ast users can also run the tests post-install and from any directory:

python -m pytest --pyargs typed_ast
@mr-c
Distribute the tests
Verified
This commit was signed with a verified signature.
mr-c Michael R. Crusoe
GPG key ID: 3C26763F6C67E6E2 Learn about signing commits
Loading status checks…
b32eb2e
@gvanrossum
gvanrossum approved these changes Feb 9, 2019
View changes
Copy link
Member

@gvanrossum gvanrossum left a comment

Sure, sounds fine.
@gvanrossum gvanrossum merged commit 33209c4 into python:master Feb 9, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mr-c mr-c deleted the mr-c:distribute_tests branch Feb 9, 2019
@stanislavlevin
Copy link

@stanislavlevin stanislavlevin commented Oct 16, 2019

@mr-c, @gvanrossum, JFYI:
in many distros docs, tests, etc. are not packaged since they are useless for most of the users. The testing in turn, is performed during the build, for doing that we don't need to move tests out to its own Python package.
So, as of this change, I have to remove tests from the distro's Python package.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gvanrossum gvanrossum

Assignees
No one assigned
Labels
CLA signed
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@mr-c @stanislavlevin @gvanrossum @the-knights-who-say-ni
You can’t perform that action at this time.