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

Remove config for Travis-CI.org #960

Merged
merged 1 commit into from
Aug 17, 2021
Merged

Conversation

michael-k
Copy link
Contributor

Doesn't work any more; you could switch to travis-ci.com though.

image

@erezsh erezsh self-requested a review August 17, 2021 09:10
Copy link
Member

@MegaIng MegaIng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason to use travis if we are using github actions? We don't need any platform specific tests right now, and travis-ci seems in general be anti Open Source right now.

@erezsh erezsh merged commit 82ad512 into lark-parser:master Aug 17, 2021
@erezsh
Copy link
Member

erezsh commented Aug 17, 2021

@michael-k Thanks!

@MegaIng I can't think of any.

@michael-k michael-k deleted the remove-travis branch August 17, 2021 09:56
@michael-k
Copy link
Contributor Author

Is there any reason to use travis if we are using github actions?

Some maintainers prefer to stick with what they know. That's the only reason why I've mentioned travis-ci.com

We don't need any platform specific tests right now

GitHub Actions lets you run workflows on Mac OS and Windows. Travis CI doesn't support Windows (iirc)

travis-ci seems in general be anti Open Source right now.

Yeah, it all went downhill after they were bought in January 2019 https://www.ideracorp.com/pressreleases/acquires-travis-ci

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.

3 participants