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

WIP: Migrate CI/CD to GitHub Action #79

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

rachaelharris
Copy link
Collaborator

@rachaelharris rachaelharris commented Apr 7, 2021

Fixing issue #28 Migration to GitHub Actions
Reviewers: @ColemanKobe @noorbuchi @enpuyou

What is the current behavior?

Current behavior is that GitHub Actions was not being used prior. Now, we have migrated to GitHub Actions.

Issue #28

What is the new behavior if this PR is merged?

The new behavior is that GitHub Actions should work properly.
The TravisCI file is still in the repository; we wanted to keep it until we were sure that this would work.
Requesting reviewers to ensure that we're on the right track.

Type of change

Please describe the pull request as one of the following:

  • Bug fix
  • Breaking change
  • New feature
  • Documentation update
  • Other

Other information

This PR has:

  • Commit messages that are correctly formatted
  • Tests for newly introduced code
  • Docstrings for newly introduced code (no new introduced code)

Developers

@zoe9078 @mceccarelli @rachaelharris

@enpuyou enpuyou changed the title moved codecov to workflows folder WIP: Migrate CI/CD to GitHub Action Apr 10, 2021
@enpuyou enpuyou added the WIP Work in progress label Apr 10, 2021
@codecov
Copy link

codecov bot commented Apr 11, 2021

Codecov Report

Merging #79 (a064fa7) into master (244b0ba) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   92.09%   92.09%           
=======================================
  Files           6        6           
  Lines         253      253           
=======================================
  Hits          233      233           
  Misses         20       20           

@corlettim
Copy link
Collaborator

Please update your branch/PR with master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants