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 some tests #77

Merged
merged 2 commits into from Jan 18, 2019
Merged

Add some tests #77

merged 2 commits into from Jan 18, 2019

Conversation

@gvanrossum
Copy link
Member

@gvanrossum gvanrossum commented Jan 18, 2019

This adds some very basic tests for ast3, and modifies .travis.yml to run them (while still also building the Linux wheels). The key here is getting some infrastructure going, we can elaborate in subsequent PRs.

@gvanrossum gvanrossum requested a review from msullivan Jan 18, 2019
@gvanrossum gvanrossum merged commit fdb351d into master Jan 18, 2019
4 checks passed
4 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@gvanrossum gvanrossum deleted the add-tests branch Jan 18, 2019
@gvanrossum
Copy link
Member Author

@gvanrossum gvanrossum commented Jan 18, 2019

Since @msullivan is out on PTO, I've landed this without waiting for a review so I can merge it into the various 3.7-related branches. Feel free to call me on the .travis.yml changes -- I don't know whether the deploy section still works and since it only triggers when we tag a release it seems cumbersome to test.

@msullivan
Copy link
Collaborator

@msullivan msullivan commented Jan 18, 2019

Test stuff looks good. Tags can be deleted, so I think it can be tested with tags and then cleaned up after.

gvanrossum added a commit that referenced this pull request Jan 23, 2019
This is a full port, following the recipe in update_process.md. I've also tried to keep the recipe up to date and improved the automation (see tools/script). I haven't cleaned up the commits. As of #77 there are a few tests that sanity-check this (though it's far from a full test suite), and they're run by Travis-CI and AppVeyor.
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.