-
Notifications
You must be signed in to change notification settings - Fork 447
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
Migrate to github actions and add codecov bot #654
Migrate to github actions and add codecov bot #654
Conversation
@techno-disaster Since you have experience with this can you kindly review? Thanks a lot!! 🎉 💯 |
| :---: | :---: | :---: | :---: | | ||
| [master](https://github.com/anitab-org/mentorship-backend/tree/master) | [![Build Status](https://travis-ci.org/systers/mentorship-backend.svg?branch=master)](https://travis-ci.org/anitab-org/mentorship-backend) | [![Codacy Badge](https://api.codacy.com/project/badge/Grade/5181ca06835045d1aea78fdd30fc37d9?branch=master)](https://www.codacy.com/app/systers/mentorship-backend) | [Production](http://systers-mentorship.eu-central-1.elasticbeanstalk.com/) | | ||
| [develop](https://github.com/anitab-org/mentorship-backend/tree/develop) | [![Build Status](https://travis-ci.org/anitab-org/mentorship-backend.svg?branch=develop)](https://travis-ci.org/anitab-org/mentorship-backend) | [![Codacy Badge](https://api.codacy.com/project/badge/Grade/5181ca06835045d1aea78fdd30fc37d9?branch=develop)](https://www.codacy.com/app/systers/mentorship-backend) | [Development](https://mentorship-backend-temp.herokuapp.com/) | | ||
![Build Status](https://github.com/anitab-org/mentorship-backend/workflows/Run%20tests/badge.svg) |
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 badge for run tests only right? Build is another badge? Also could you get a badge for codecov?
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.
Are we not using Codacy anymore 🤔 I actually am not sure cc @mayburgos
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.
Hey @isabelcosta the travis script just runs the tests and then publishes on aws as I directly converted the travis script into gh actions 😅
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.
The codacy thing I am not sure about @isabelcosta 😅 I figured I didn't see the test running so we might not be using it but not sure
deploy.sh
Outdated
@@ -1,49 +0,0 @@ | |||
#!/usr/bin/env bash |
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.
Not sure if we should remove this file because the Herokuhoosting should be a temporary thing. Perhaps we could keep it and leave a comment as this is not being currently used 🤔
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.
Yeah sure will edit @isabelcosta ✌️
b17c5ec
to
cb5ba32
Compare
cb5ba32
to
aaf2427
Compare
@isabelcosta Have added the codecov badge and the deploy.sh file back 😅 also just asking here we have the codedcov application setup on anitab-org right (under the settings tab in github under applications) ? I am assuming that since we are using codecov for the flutter app it has been already setup 😅 |
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.
Looks good @Tlazypanda :)
Looking forward to see this working.
@Tlazypanda can you ask for additional reviews on Zulip, after we have one more I am more than happy to merge this |
I will merge this and see how it goes. |
chore: migrate to github actions and add codecov bot (anitab-org#654)
Description
Migrated to github actions and eliminated travis related files. Added support for codecov in gh action script along with codecov bot to comment on every pr. Readme edited to add gh actions status badge. Removed travis script (github workflow is used now) and deploy.sh script (since aws services no longer in use )
Fixes #659
Type of Change:
Code/Quality Assurance Only
How Has This Been Tested?
Tested on my fork of mentorship-backend.
Checklist:
Code/Quality Assurance Only