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

Features/travis ci #154

Merged
merged 5 commits into from
Jun 2, 2016
Merged

Features/travis ci #154

merged 5 commits into from
Jun 2, 2016

Conversation

birgits
Copy link
Member

@birgits birgits commented May 10, 2016

No description provided.

@gplssm
Copy link
Contributor

gplssm commented May 10, 2016

Tests pass with if Python 3.3 is omitted. Shall we exclude Python 3.3 from test cases?

@gplssm
Copy link
Contributor

gplssm commented May 10, 2016

After @gnn fixes #155 re-add Python 3.3 to .travis.yml

@uvchik
Copy link
Member

uvchik commented May 12, 2016

@gplssm Will you merge it?

@gplssm
Copy link
Contributor

gplssm commented May 12, 2016

Yes, I'll do it, when @gnn gives his OK.

@uvchik uvchik assigned gnn and unassigned gplssm May 12, 2016
@uvchik
Copy link
Member

uvchik commented May 12, 2016

To reduce the server load, we could delete the push-test. I think the PR-test is enough.

@uvchik uvchik added this to the v0.0.9 milestone May 12, 2016
@gplssm
Copy link
Contributor

gplssm commented May 12, 2016

I disabled "Build pushes" only "Build PR" is active. If you uncomfortable with this, we can reconfigure it...

@uvchik
Copy link
Member

uvchik commented May 23, 2016

I disabled "Build pushes" only "Build PR" is active. If you uncomfortable with this, we can reconfigure it...

@gplssm : Where did you do it? Did you push it? I do not see a commit.

@gplssm
Copy link
Contributor

gplssm commented May 23, 2016

I disabled in the config on the travis website which is not visible in the commit history.

@uvchik
Copy link
Member

uvchik commented May 23, 2016

But there are still two checks (continuous-integration/travis-ci/push and continuous-integration/travis-ci/pr) or do I misunderstand something?

@gplssm
Copy link
Contributor

gplssm commented Jun 2, 2016

We still see 2 tests here because my changes in the settings are older then the last commit in this PR. Have a look at PR #159, where only one test run is shown.

@gplssm gplssm merged commit d8564fd into dev Jun 2, 2016
@uvchik uvchik deleted the features/travis-CI branch June 2, 2016 20:30
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.

4 participants