Skip to content

Fix CI build#2

Merged
msis merged 2 commits intomsis:masterfrom
russkel:fix-ci
Apr 20, 2020
Merged

Fix CI build#2
msis merged 2 commits intomsis:masterfrom
russkel:fix-ci

Conversation

@russkel
Copy link
Copy Markdown

@russkel russkel commented Apr 16, 2020

Fixes Travis building and testing for Linux and OSX. I don't have any experience with appveyor or building this on Windows so I will leave it for someone else.

@russkel
Copy link
Copy Markdown
Author

russkel commented Apr 16, 2020

Ah, travis CI isn't activated for PRs. I'll get travis working on my own repository.

@russkel russkel force-pushed the fix-ci branch 2 times, most recently from e530add to bc0239b Compare April 16, 2020 07:01
@msis
Copy link
Copy Markdown
Owner

msis commented Apr 16, 2020

Thanks Russ!

I'm not sure how to activate Travis for PRs, please advise.

@russkel
Copy link
Copy Markdown
Author

russkel commented Apr 16, 2020

No problem @msis. To activate travis for PRs, log onto to Travis, go to settings for the repository and then make sure "Build pushed PRs" is turned on:

image

Also: https://travis-ci.com/github/russkel/python-moos/jobs/319908963 these two tests are genuinely failing. Any idea why?

You also use your own branch of core-moos for CI, which we probably shouldn't need to do. Can we rebase that branch off the master core-moos and upstream the changes? It looks like mostly Appveyor mods, so I will change the travis one to use the official core-moos branch.

@russkel russkel force-pushed the fix-ci branch 6 times, most recently from e09bba6 to 96bca04 Compare April 17, 2020 04:54
@russkel
Copy link
Copy Markdown
Author

russkel commented Apr 17, 2020

Another case of it happening here https://travis-ci.com/github/russkel/python-moos/jobs/320319760

@russkel
Copy link
Copy Markdown
Author

russkel commented Apr 17, 2020

Alright, done. This should be good to merge. Results can be seen here: https://travis-ci.com/github/russkel/python-moos

The failures are due to test failures, which was the point of this PR.

@msis
Copy link
Copy Markdown
Owner

msis commented Apr 20, 2020

No problem @msis. To activate travis for PRs, log onto to Travis, go to settings for the repository and then make sure "Build pushed PRs" is turned on:

image

Also: https://travis-ci.com/github/russkel/python-moos/jobs/319908963 these two tests are genuinely failing. Any idea why?

You also use your own branch of core-moos for CI, which we probably shouldn't need to do. Can we rebase that branch off the master core-moos and upstream the changes? It looks like mostly Appveyor mods, so I will change the travis one to use the official core-moos branch.

That's what I have in the settings too.
I will go ahead and merge so we can check #3.

@msis msis merged commit dd2ade7 into msis:master Apr 20, 2020
@russkel
Copy link
Copy Markdown
Author

russkel commented Apr 24, 2020

Hmm, okay there must be something I am missing. It looks like travis is building the PRs: https://travis-ci.org/github/msis/python-moos/pull_requests

I think there is an option in GitHub, I'll see if I can find it.

@russkel
Copy link
Copy Markdown
Author

russkel commented Apr 24, 2020

russkel#1

It's working on my repository. The only thing I can think of is due to the age of this repository maybe Travis CI does not have Github Apps integration: https://blog.travis-ci.com/2018-05-07-announcing-support-for-github-checks-api-on-travis-ci-com

Instructions on how to migrate are in that blog post.

@russkel russkel deleted the fix-ci branch November 21, 2020 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants