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: Add test workflow #32

Merged
merged 1 commit into from
Feb 14, 2022
Merged

ci: Add test workflow #32

merged 1 commit into from
Feb 14, 2022

Conversation

joeyparrish
Copy link
Member

Although there aren't any real tests (see #31), we can run the linter
on new PRs. When we add tests later, the workflow will activate them
through npm test.

Although there aren't any real tests (see #31), we can run the linter
on new PRs.  When we add tests later, the workflow will activate them
through npm test.
@joeyparrish joeyparrish added the type: CI An issue with our continuous integration tests label Feb 12, 2022
@joeyparrish joeyparrish requested a review from theodab February 12, 2022 01:07
@joeyparrish
Copy link
Member Author

Looks like I need to fix the line-ending style in .gitattributes to make the linter checks pass on Windows. I'll go ahead and merge this, though, and follow-up on that.

@joeyparrish joeyparrish merged commit a40257e into shaka-project:main Feb 14, 2022
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Sep 2, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated type: CI An issue with our continuous integration tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants