Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Feat/migrate to gh actions #53

Merged
merged 10 commits into from
Nov 17, 2020
Merged

Feat/migrate to gh actions #53

merged 10 commits into from
Nov 17, 2020

Conversation

tunderwood
Copy link
Contributor

No description provided.


steps:
- uses: actions/checkout@v2
- name: Set up Python ${{ matrix.python-version }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this work to set up both version of Python?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it looks at the "strategy:matrix:python-version" field under Build and then creates separate workflows for each one.

uses: codecov/codecov-action@v1
with:
file: coverage.xml # optional
fail_ci_if_error: true # optional (default = false)
Copy link
Contributor

Choose a reason for hiding this comment

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

Will this send a Slack notification if any tests fail?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had actually forgotten to put the slack config in so thanks for catching that.

This will notify us in slack anytime the master build fails (due to failed tests, failed coverage upload, or anything else)

@tunderwood tunderwood merged commit 7bd92a8 into master Nov 17, 2020
@tunderwood tunderwood deleted the feat/migrat_to_GH_actions branch November 17, 2020 21:08
tunderwood added a commit that referenced this pull request Nov 18, 2020
*Migrate to GH Actions
tunderwood added a commit that referenced this pull request Nov 18, 2020
*Migrate to GH Actions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants