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

ci: migrate to GH actions #34

Merged
merged 1 commit into from
Jul 13, 2023
Merged

ci: migrate to GH actions #34

merged 1 commit into from
Jul 13, 2023

Conversation

agilgur5
Copy link
Owner

Summary

Migrate over to GH Actions from Travis

Details

  • mostly duplicate what I have in react-signature-canvas

    • update LTS Node versions to 18.x and 20.x
    • comment out linting step for now as ts-standard hasn't been configured in this repo
  • add tsc and test:coverage scripts for CI

    • had to specify --noEmit for now, the whole build system will need to be revamped later
  • replace Travis badge with GH Actions badge

- mostly duplicate what I have in [react-signature-canvas](https:/agilgur5/react-signature-canvas/blob/main/.github/workflows/ci.yml)
  - update LTS Node versions to 18.x and 20.x
  - comment out linting step for now as `ts-standard` hasn't been configured in this repo

- add `tsc` and `test:coverage` scripts for CI
  - had to specify `--noEmit` for now, the whole build system will need to be revamped later

- replace Travis badge with GH Actions badge
Copy link
Owner Author

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

awesome, tests pass on the new GH Actions CI!

had to modify branch protection for the new checks and had to reactivate Codecov (which is glitching on the master -> main branch rename), but otherwise seems to be working 🙂

thankfully was pretty straightforward when copying existing code!

@agilgur5 agilgur5 merged commit def64c8 into main Jul 13, 2023
6 checks passed
@agilgur5 agilgur5 deleted the ci-gh-actions branch July 13, 2023 02:32
@agilgur5 agilgur5 added the kind: internal Changes only affect the internal API label Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: internal Changes only affect the internal API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant