-
Notifications
You must be signed in to change notification settings - Fork 288
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
add package-link file to storefront #745
Conversation
Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.7. - [Release notes](https://github.com/isaacs/ini/releases) - [Commits](npm/ini@v1.3.5...v1.3.7) Signed-off-by: dependabot[bot] <[email protected]>
a8a2728
to
ba7e068
Compare
Signed-off-by: Erik Kieckhafer <[email protected]>
ba7e068
to
65c6ecd
Compare
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
@kieckhafer We should add the |
Signed-off-by: Erik Kieckhafer <[email protected]>
@mikemurray done. |
@@ -0,0 +1,23 @@ | |||
#!/usr/bin/env bash |
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 this file needs execute permissions.
Not sure why the push is failing. |
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Akarshit Wal <[email protected]>
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.
👍
Impact: minor
Type: chore
Adds
package-link
andpackage-unlink
, similar to we have in theapi
project, to allow for dev linking of outside npm packages.We need to update the base image to use
12.14.1-v3
instead of12.14.1-v1
as this version installsyalc
inside the container, which is required for this linking.TESTING:
bin/package-link @reactioncommerce/components ../reaction-component-library/package
bin/package-unlink @reactioncommerce/components