-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: Add basic Github Actions workflow and clean up index #2
feat: Add basic Github Actions workflow and clean up index #2
Conversation
'no-custom-fontWeight': noCustomFontWeight, | ||
}, | ||
export const rules = { | ||
'no-custom-fontWeight': noCustomFontWeight, |
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.
I think this should be fine but eslint is weird
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.
I'll revert if it breaks stuff. I think tsc
will transform this
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.
Thanks!
NPM_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
run: | | ||
npm config set //registry.npmjs.org/:_authToken ${NPM_TOKEN} | ||
npm publish --access public |
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.
BUN
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.
Should make the dependency install fast. We'll see how it goes
@bnussman-akamai you also wanna bump the version? |
@abailly-akamai I'll bump after I merge this! |
No description provided.