-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Issue #16830: infra: migrate from Travis CI to GitHub Actions in configuration files #16838
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
base: master
Are you sure you want to change the base?
Conversation
5558502
to
fa8d86d
Compare
f5d711a
to
fa8d86d
Compare
…onfiguration files
fa8d86d
to
476a960
Compare
@romani I have fixed Ci which was failing before, now all CI is green - Good to go? |
I lost a context why we removed Travis, it works, and does not bother us. |
Like we not use Travis anymore in ci checks for PR, So better to remove it's context from repo. |
@romani not heard back anything from you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
item:
|
||
after_success: | ||
- ./.ci/travis.sh run-command-after-success | ||
- ./.ci/travis.sh deploy-snapshot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the only actiion that is left Travis
we need to convert this to github action that executes only on master branch after merge of any commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #16830
Closes #16837
I have tried to remove or update all Travis ci reference through out the repo.
At some places switched travis ci to github actions.
Note: I have removed complete 2 files as it was not needed both where of Travis CI.
Build status
