Skip to content
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

V8 release #239

Merged
merged 15 commits into from
Jun 29, 2017
Merged

V8 release #239

merged 15 commits into from
Jun 29, 2017

Conversation

broccolini
Copy link
Member

@broccolini broccolini commented Jun 28, 2017

~~Small readme updates and publishes v7 🎉 ~~

Okay scratch that! We found during testing that all the imports were in a different order to github.com and were causing some override issues. Updated this pr to fix that!

Also in this pr:

  • added force publish npm script for convenience
  • renamed packages to modules

cc @primer/ds-core

Copy link
Contributor

@shawnbot shawnbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IT'S ALIVE!

- utilities should be last so that they override components
- matching the rest of the order to github.com so as to avoid potential
issues with the cascade
@import "primer-truncate/index.scss";

// Utilities always go last so that they can trump components
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"trump" => "override" ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure

@broccolini broccolini changed the title V7 release V8 release Jun 29, 2017
@shawnbot
Copy link
Contributor

Still gets my 👍 . I especially appreciate renaming packages to modules, which helps with autocomplete.

@broccolini broccolini merged commit 454a884 into master Jun 29, 2017
@broccolini broccolini deleted the v7_release branch June 29, 2017 23:25
@shawnbot
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants