Skip to content

Conversation

@KHVBui
Copy link
Collaborator

@KHVBui KHVBui commented Aug 1, 2022

No description provided.

@KHVBui KHVBui self-assigned this Aug 1, 2022
@KHVBui KHVBui requested a review from minhngo3818 August 1, 2022 05:30
@KHVBui KHVBui added not ready PR PR is not ready to be merged CI/CD continuous integration and continuous deployment labels Aug 1, 2022
Copy link
Collaborator

@minhngo3818 minhngo3818 left a 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.

@KHVBui KHVBui changed the title Create Github Actions file to test builds Create Github Actions file to test builds before merging Aug 1, 2022
@KHVBui KHVBui closed this Aug 1, 2022
@KHVBui KHVBui force-pushed the Build-Test-for-CI branch from 0bc816d to 96dbd71 Compare August 1, 2022 17:10
@KHVBui KHVBui reopened this Aug 1, 2022
@KHVBui
Copy link
Collaborator Author

KHVBui commented Aug 1, 2022

Going to need a Lint fix PR before merging this PR

@KHVBui KHVBui force-pushed the Build-Test-for-CI branch from ffdf27d to 0711910 Compare August 1, 2022 23:49
- Incorporate eslint, prettier, and windows/mac builds
- Fail-fast is set to false so tests on all OS's are run before stopping
@KHVBui KHVBui force-pushed the Build-Test-for-CI branch from 0711910 to 96c9038 Compare August 2, 2022 04:32
@KHVBui KHVBui requested a review from minhngo3818 August 2, 2022 05:44
@KHVBui
Copy link
Collaborator Author

KHVBui commented Aug 2, 2022

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

@minhngo3818
Copy link
Collaborator

Sweet! Once this PR merged, we're going use this workflow to check every PRs.

@KHVBui
Copy link
Collaborator Author

KHVBui commented Aug 2, 2022

Can you make the accepting review to merge? xD

@KHVBui KHVBui removed the not ready PR PR is not ready to be merged label Aug 2, 2022
@minhngo3818 minhngo3818 merged commit f6b1180 into main Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD continuous integration and continuous deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants