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 CI builds disabled due to negative credit balance #446

Closed
ben-heil opened this issue Nov 29, 2021 · 3 comments · Fixed by #447
Closed

Travis CI builds disabled due to negative credit balance #446

ben-heil opened this issue Nov 29, 2021 · 3 comments · Fixed by #447

Comments

@ben-heil
Copy link

The Travis-CI build badge in the README reports an error, possibly due to the travis-ci.org -> .com switch. It looks like the .travis.yml file is only included for legacy reasons, so should the README badge be removed?

@dhimmel
Copy link
Member

dhimmel commented Nov 29, 2021

From the README:

[![Travis Build Status](https://travis-ci.com/manubot/rootstock.svg?branch=main)](https://travis-ci.com/manubot/rootstock)

So it looks like we've already fully switched from travis-ci.org -> .com.

When I login to Travis and go to https://app.travis-ci.com/github/manubot/rootstock, I see:

Builds have been temporarily disabled for public repositories due to a negative credit balance. Please go to the Plan page to replenish your credit balance or alter your Consume paid credits for OSS setting.

I'm wondering whether we should remove Travis support entirely, since it's a lot of support effort and now requires payment without any unique benefit. @agitter wdyt?

@dhimmel dhimmel changed the title Badge reporting build error Travis CI builds disabled due to negative credit balance Nov 29, 2021
@agitter
Copy link
Member

agitter commented Nov 29, 2021

it's a lot of support effort and now requires payment without any unique benefit

Exactly. I don't see any reason to use it for a new manuscript. I'm okay removing Travis support entirely. We would need to update SETUP.md in several places to note it is no longer supported and remove the setup instructions.

Existing projects using Travis CI could be affected if we delete .travis.yml and they update rootstock without looking at the merge carefully. We could help users restore that file if they get stuck. I can't think of a better way to avoid removing the file if it is in use.

dhimmel added a commit to dhimmel/manubot-rootstock that referenced this issue Nov 29, 2021
@dhimmel
Copy link
Member

dhimmel commented Nov 29, 2021

Existing projects using Travis CI could be affected if we delete .travis.yml and they update rootstock without looking at the merge carefully. We could help users restore that file if they get stuck

Yeah! Happy to help legacy users upgrade to GitHub Actions. I imagine there are not too many active manuscripts still using Travis, especially now that it's a paid service.

dhimmel added a commit that referenced this issue Dec 3, 2021
dhimmel added a commit to dhimmel/manubot that referenced this issue Jan 19, 2022
dhimmel added a commit to dhimmel/manubot that referenced this issue Jan 24, 2022
dhimmel added a commit to manubot/manubot that referenced this issue Jan 24, 2022
merges #318

Python 3.6 officially reached end-of-life on 2021-12-23.

Remove travis CI config
refs manubot/rootstock#446

GitHub Actions YAML: quote version strings
Python 3.10 gets interpreted as 3.1 otherwise.

Update README for Python 3.10

universal_newlines --> text
ploegieku added a commit to ploegieku/2023-functional-homology-paper that referenced this issue Aug 6, 2024
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 a pull request may close this issue.

3 participants