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

Improve Github workflows #39

Merged
merged 2 commits into from
Mar 13, 2022

Conversation

elyse0
Copy link
Contributor

@elyse0 elyse0 commented Mar 12, 2022

Two improvements are made:

  • Run publish action only on the original repository

This avoids running the action on forks and getting a failed run.

image

  • Add test suite action

This action runs on every push to any branch and also pull requests, to make sure tests are still passing.

This will avoid executing the action in forks
This action will execute the test suite on every push to any branch or
pull request.
@elyse0 elyse0 changed the base branch from main to develop March 12, 2022 22:43
@LuisEnMarroquin LuisEnMarroquin merged commit 35b5326 into LuisEnMarroquin:develop Mar 13, 2022
@LuisEnMarroquin
Copy link
Owner

Good catch thanks 🤗

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.

2 participants