Skip to content

Fixes #4553 : Migrate CI/CD Pipeline from TravisCI to GitHub Actions #4576

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 7 commits into
base: master
Choose a base branch
from

Conversation

lazyperson1020
Copy link
Contributor

This PR adds github ci/cd workflows to existing TravisCI configuration, maintaining all existing pipeline functionality.

The new GitHub Actions workflow includes:

  • Code linting with flake8
  • Docker build process
  • Frontend testing with Karma
  • Backend testing with pytest and coverage reporting
  • Automatic deployment on master branch merges

Required secrets:

  • DOCKER_USERNAME - Docker registry username
  • DOCKER_PASSWORD - Docker registry password
  • COVERALLS_REPO_TOKEN - Token for Coveralls coverage reporting
  • SSH_PRIVATE_KEY - Deployment SSH key

Copy link

codecov bot commented Apr 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.53%. Comparing base (96968d6) to head (9cc2a08).
Report is 1151 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4576      +/-   ##
==========================================
- Coverage   72.93%   69.53%   -3.40%     
==========================================
  Files          83       20      -63     
  Lines        5368     3591    -1777     
==========================================
- Hits         3915     2497    -1418     
+ Misses       1453     1094     -359     

see 65 files with indirect coverage changes

see 65 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ae26ff...9cc2a08. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@RishabhJain2018
Copy link
Member

Hi @lazyperson1020 , Can you please update your script with the latest travis.yml in the repo?

@lazyperson1020
Copy link
Contributor Author

@RishabhJain2018 I have made some changes

@lazyperson1020
Copy link
Contributor Author

@RishabhJain2018 any update on this sir?

Copy link
Member

@RishabhJain2018 RishabhJain2018 left a comment

Choose a reason for hiding this comment

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

Hey @lazyperson1020 ,

This is the latest Travis YML: https://github.com/Cloud-CV/EvalAI/blob/master/.travis.yml

Can you please update the new file so that we can run it in the same way i.e. running code quality in docker containers and use the pylint config from .pylintrc. There are other changes too, please do those as well.

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.

2 participants