-
Notifications
You must be signed in to change notification settings - Fork 184
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
Koa 5884 migrating packages #2908
Conversation
fb56fec
to
6289fd9
Compare
@@ -16,7 +16,7 @@ | |||
* limitations under the License. | |||
*/ | |||
|
|||
@import '~bpk-mixins/index.scss'; | |||
@import '../../packages/bpk-mixins/index.scss'; |
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 guess all of these imports were automatically updated by the code editor? 😄
c23e5d6
to
aced684
Compare
f979901
to
8b03f04
Compare
Visit https://backpack.github.io/storybook-prs/2908 to see this build running in a browser. |
8ead54b
to
702c881
Compare
702c881
to
e475242
Compare
Visit https://backpack.github.io/storybook-prs/2908 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/2908 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/2908 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/2908 to see this build running in a browser. |
.gitignore
Outdated
/dist-storybook | ||
/dist-sassdoc |
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 now added twice 😄
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.
LGTM!
Visit https://backpack.github.io/storybook-prs/2908 to see this build running in a browser. |
This reverts commit 7156ceb.
Migrating
bpk-mixins
andbpk-stylesheets
to Backpack WebThe benefit of doing so are:
For Backpack itself
@skyscanner/backpack-web
andbpk-mixins
, and evenbpk-stylesheets
exist together so benefits to consuming projects tooFor consumers
@skyscanner/backpack-web
, instead of having to install separately and the risk of having a version defined of mixins/stylesheets whilst Backpack web had older versions defined.Remember to include the following changes:
README.md
(If you have created a new component)README.md