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.

Sign up for GitHub

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

Jump to bottom

Add CI GitHub action #339

Merged
merged 2 commits into from
Oct 17, 2021
Merged

Add CI GitHub action #339

merged 2 commits into from
Oct 17, 2021

Conversation

andy128k
Copy link
Collaborator

@andy128k andy128k commented Oct 17, 2021 •
edited
Loading

  • Changes in .gitattributes are needed for a prettier/eslint
  • Nodejs 12 and 14 are supported. Nodejs 16 is not supported because of broken fibers. This may be done later.
  • Actions "in action" are here :)

@andy128k andy128k mentioned this pull request Oct 17, 2021
Merged
@alexander-akait
Copy link
Member

@andy128k let's remove config for travis and appveyor, big thanks

@andy128k
Copy link
Collaborator Author

andy128k commented Oct 17, 2021 •
edited
Loading

@alexander-akait done

@alexander-akait alexander-akait merged commit 70ea4e4 into itgalaxy:master Oct 17, 2021
@alexander-akait
Copy link
Member

@andy128k thanks

Nodejs 12 and 14 are supported. Nodejs 16 is not supported because of broken fibers. This may be done later.

Will be great to fix it too

@andy128k andy128k deleted the github-actions branch October 17, 2021 20:31
@andy128k
Copy link
Collaborator Author

Created a ticket #340 to not to forget about 16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andy128k @alexander-akait