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

Closed

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.

Copy link
Contributor Author

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.

any idea how to convert this same to github actions? is there any direct configure or setup for same available on github? so that we really not to write whole file again.

Copy link
Member

Choose a reason for hiding this comment

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

too much to explain, not easy
please close this PR, you can come back to this issue later on as you get more experience, lets practice of more light updates first.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

got it

@Brijeshthummar02 Brijeshthummar02 deleted the Travis-CI branch May 17, 2025 14:12
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