-
Couldn't load subscription status.
- Fork 16
Create Github Actions file to test builds before merging #35
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Git action needs configuring eslint and prettier in test build in order to pass. Please add windows and mac to run test.
0bc816d to
96dbd71
Compare
|
Going to need a Lint fix PR before merging this PR |
ffdf27d to
0711910
Compare
- Incorporate eslint, prettier, and windows/mac builds - Fail-fast is set to false so tests on all OS's are run before stopping
0711910 to
96c9038
Compare
- We don't have any tests, and we only need to test if the app will build here
|
PHEW I finally got the workflow to work. There was this fun bug where it didn't work on the windows build since windows uses a different eol character, but adding a gitattributes file to do eol normalization fixed it |
|
Sweet! Once this PR merged, we're going use this workflow to check every PRs. |
|
Can you make the accepting review to merge? xD |
No description provided.