-
Notifications
You must be signed in to change notification settings - Fork 830
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
Creating a custom action for building all javascript based octicons packages #279
Conversation
@@ -0,0 +1,3 @@ | |||
#!/usr/bin/env node | |||
|
|||
console.log("Create a build/data.json from the exported assets") |
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.
TODO: Right now the figma-action is specific to octicons and is still building the build/data.json for us. I'd like to make it more generic and do the conversion work here. Until then
@@ -17,9 +17,8 @@ | |||
"url": "https://github.com/primer/octicons/issues" | |||
}, | |||
"scripts": { | |||
"publish": "../../script/notify success", |
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 we keep these in here until the publish stuff is merged in?
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.
Actions notifies for us now. 🥇
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.
Just left one comment, otherwise 👍
This is a common action that builds any npm based project.
octicons_node
andocticons_react
The steps it does is
@shawnbot @emplums