Skip to content

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Brijeshthummar02
Copy link
Contributor

@Brijeshthummar02 Brijeshthummar02 commented Apr 12, 2025

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
Screenshot 2025-04-12 214020

@Brijeshthummar02 Brijeshthummar02 changed the title ci #16830: migrate from Travis CI to GitHub Actions in configuration files Issue #16830: migrate from Travis CI to GitHub Actions in configuration files Apr 12, 2025
@Brijeshthummar02 Brijeshthummar02 changed the title Issue #16830: migrate from Travis CI to GitHub Actions in configuration files Issue #16830: infra: migrate from Travis CI to GitHub Actions in configuration files Apr 13, 2025
@Brijeshthummar02
Copy link
Contributor Author

@romani I have fixed Ci which was failing before, now all CI is green - Good to go?

@Brijeshthummar02
Copy link
Contributor Author

@romani I have fixed Ci which was failing before, now all CI is green - Good to go?

@romani may I get your attention please.

@romani
Copy link
Member

romani commented Apr 16, 2025

I lost a context why we removed Travis, it works, and does not bother us.

@Brijeshthummar02
Copy link
Contributor Author

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.
I thought good to clean up. What you suggest should we accept it or let the PR Closed.

@Brijeshthummar02
Copy link
Contributor Author

@romani not heard back anything from you.

Copy link
Member

@romani romani left a 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
Copy link
Member

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.

Copy link
Member

Choose a reason for hiding this comment

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

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.

Documentation Outdated CI Management Page Still Refers to Travis CI
2 participants