Skip to content
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

Merged
merged 2 commits into from
Jul 16, 2020

Conversation

Tlazypanda
Copy link
Contributor

@Tlazypanda Tlazypanda commented Jul 5, 2020

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

Code/Quality Assurance Only

  • Bug fix (non-breaking change which fixes an issue)
  • This change requires a documentation update (software upgrade on readme file)
  • New feature (non-breaking change which adds functionality pre-approved by mentors)

How Has This Been Tested?

Tested on my fork of mentorship-backend.

Checklist:

  • My PR follows the style guidelines of this project
  • I have performed a self-review of my own code or materials
  • I have commented my code or provided relevant documentation, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

Code/Quality Assurance Only

  • My changes generate no new warnings

@Tlazypanda
Copy link
Contributor Author

@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)
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 badge for run tests only right? Build is another badge? Also could you get a badge for codecov?

Copy link
Member

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

Copy link
Contributor Author

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 😅

Copy link
Contributor Author

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
Copy link
Member

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 🤔

Copy link
Contributor Author

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 ✌️

@Tlazypanda Tlazypanda force-pushed the migrate_github_actions branch from b17c5ec to cb5ba32 Compare July 15, 2020 10:24
@Tlazypanda Tlazypanda force-pushed the migrate_github_actions branch from cb5ba32 to aaf2427 Compare July 15, 2020 10:48
@Tlazypanda
Copy link
Contributor Author

@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 😅

@Tlazypanda Tlazypanda requested a review from isabelcosta July 15, 2020 10:58
Copy link
Member

@isabelcosta isabelcosta left a 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.

@isabelcosta
Copy link
Member

@Tlazypanda can you ask for additional reviews on Zulip, after we have one more I am more than happy to merge this

@isabelcosta isabelcosta changed the title WIP: migrate to github actions and add codecov bot Migrate to github actions and add codecov bot Jul 16, 2020
@isabelcosta
Copy link
Member

I will merge this and see how it goes.

@isabelcosta isabelcosta merged commit 01e5cec into anitab-org:develop Jul 16, 2020
robotjellyzone added a commit to robotjellyzone/mentorship-backend that referenced this pull request Jul 17, 2020
chore: migrate to github actions and add codecov bot (anitab-org#654)
b-thebest pushed a commit to b-thebest/mentorship-backend that referenced this pull request Mar 2, 2021
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.

Migrate to Github actions and add codecov bot
2 participants