-
Notifications
You must be signed in to change notification settings - Fork 12
removed immutablejs for handling state immutability #37
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
base: master
Are you sure you want to change the base?
removed immutablejs for handling state immutability #37
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.
lets add a CD pipeline too
.github/workflows/npm-publish.yml
Outdated
@@ -0,0 +1,33 @@ | |||
# This workflow will run tests using node and then publish a package to GitHub Packages when a release is created |
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.
remove this
.github/workflows/npm-publish.yml
Outdated
# This workflow will run tests using node and then publish a package to GitHub Packages when a release is created | ||
# For more information see: https://help.github.com/actions/language-and-framework-guides/publishing-nodejs-packages | ||
|
||
name: Node.js Package |
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.
name it correctly
.github/workflows/npm-publish.yml
Outdated
- master | ||
|
||
jobs: | ||
build: |
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.
this is just installing dependencies right, shouldn't be called build.
Also, we don't really need to install dependencies or build in this case.
.github/workflows/npm-publish.yml
Outdated
with: | ||
node-version: 14 | ||
registry-url: https://registry.npmjs.org/ | ||
- run: yarn install |
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.
we're installing again here
9e3ff11
to
f3288e1
Compare
Ticket Link
Related Links
Description
Steps to Reproduce / Test
GIF's