Skip to content

Conversation

@RustyBower
Copy link
Contributor

@RustyBower RustyBower commented May 27, 2021

Description

Migrates sopel CI/CD from Travis CI to GitHub Actions.

Checklist

  • I have read CONTRIBUTING.md
  • I can and do license this contribution under the EFLv2
  • No issues are reported by make qa (runs make quality and make test)
  • I have tested the functionality of the things this change touches

@dgw dgw changed the title Gha GitHub actions setup ci: switch to GitHub Actions May 27, 2021
@dgw dgw added this to the 8.0.0 milestone May 27, 2021
@RustyBower RustyBower force-pushed the gha-github-actions-setup branch 8 times, most recently from 69e6964 to 53d4637 Compare May 27, 2021 03:20
@dgw
Copy link
Member

dgw commented May 27, 2021

Note to self: Netlify branch previews fail because of dependencies that have changed since this branch diverged from master, and can be safely ignored. Once merged, it will work fine; I just tested it.

@dgw dgw force-pushed the gha-github-actions-setup branch 2 times, most recently from a574ec1 to 41997d5 Compare May 30, 2021 01:54
dgw and others added 2 commits May 29, 2021 22:05
* Moved over linting/style-check/test actions
* Reimplemented Coveralls data upload
* Removed unnecessary .gitignore entries

Co-authored-by: Rusty Bower <[email protected]>
We obviously don't need a `make travis` recipe any more.

Other little things that nobody needs or wants could also be removed,
plus some of the targets were missing from the list of phonies.
@dgw dgw force-pushed the gha-github-actions-setup branch from cffa5d5 to 4eec229 Compare May 30, 2021 03:06
Copy link
Member

@dgw dgw left a comment

Choose a reason for hiding this comment

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

I wrote part of this, so it's no surprise that I approve, but here. 😸

@dgw dgw merged commit 9c92df6 into master May 30, 2021
@dgw dgw deleted the gha-github-actions-setup branch May 30, 2021 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants