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

[travis] Auto-conf deps=high matrix line #16858

Closed
wants to merge 1 commit into from

Conversation

nicolas-grekas
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

This generalize what we did for testing while working on major version:

  • on master, the deps=high line checks out the last "number" branch for testing
  • legacy tests are skipped when the last X.Y series is tested with master (X+1).0

@nicolas-grekas
Copy link
Member Author

Travis config is ready. Once merged on 3.0, I'll cherry-pick the PR on 2.3 (doing it this way reduces conflicts)

@nicolas-grekas
Copy link
Member Author

@stof , willing to review this one?

@jakzal
Copy link
Contributor

jakzal commented Dec 10, 2015

@nicolas-grekas .travis.yml is really complex. It's not a subject of this PR, but what do you think of extracting most of its logic to a bash script? Could be more readable.

@nicolas-grekas
Copy link
Member Author

PR rebased
@jakzal I'm convinced it would really help... Here, everything is in just one place, just read the file to understand how travis runs tests (even if it's complex I agree)...

@nicolas-grekas
Copy link
Member Author

Status: needs work
Testing 2.8 should skip legacy tests, this is not the case currently

@nicolas-grekas
Copy link
Member Author

Green on 3.0, now testing on 2.3: #16974

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants