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

Use GitHub Actions #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Use GitHub Actions #1

wants to merge 1 commit into from

Conversation

chrismcdev
Copy link
Owner

Context

In order to prevent errors from being merged and create a culture of writing up tests for new features, CI must be used to run the tests. GitHub actions was selected as the best option due to its availability and integration with GitHub.

Changes

  • Create .github/workflows/main.yaml workflow. ✨
    • Triggered on commits merged into main and pull requests merging into main.
    • Run Jest tests via yarn test command.

Consequences

Tests will now run on future pull requests and will prevent merging if errors occur.

@chrismcdev
Copy link
Owner Author

@chris-everest here's an example of a typical pull request I'd make with the headings context, changes and consequences.

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.

1 participant