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

Dev branch for new release #243

Merged
merged 9 commits into from Jul 10, 2017
Merged

Dev branch for new release #243

merged 9 commits into from Jul 10, 2017

Conversation

broccolini
Copy link
Member

@broccolini broccolini commented Jul 3, 2017

In this pr:

  • update repo urls from packages to modules
  • update dev and rc publish scripts to --skip-git since we're unlikely to want them to publish tags, and I removed the script for regular publishing and skipping git as in the majority of cases we'll want to publish git tags with releases

Update ❗️

I noticed some odd quirks while working on the new style guide and discovered that we are not pulling in the latest version of primer-base in primer-core and primer-css. We're currently pointing to 0.4.0 which is the version before we added our own normalize to base 😱 . This may explain some of the odd issues we're finding on GitHub.com.

I'll open a pr in github/github to bump the version which I'll ship asap on Monday as soon as both pr's are reviewed.

shawnbot
shawnbot previously approved these changes Jul 5, 2017
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.

👍

@broccolini broccolini dismissed shawnbot’s stale review July 9, 2017 23:09

This pr wasn't ready for review, I've made major updates since so needs to be reviewed again.

@broccolini
Copy link
Member Author

Published 9.0.0-rc release and opened a pr on github to test: https://github.com/github/github/pull/75583

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.

🤗

 - primer-alerts@1.1.8
 - primer-avatars@1.0.2
 - primer-base@1.1.5
 - primer-blankslate@1.0.2
 - primer-box@2.1.8
 - primer-breadcrumb@1.0.2
 - primer-buttons@2.0.6
 - primer-cards@0.1.8
 - primer-core@6.0.0
 - primer-css@9.0.0
 - primer-forms@1.0.13
 - primer-labels@1.1.6
 - primer-layout@1.0.5
 - primer-markdown@3.3.13
 - primer-marketing-support@1.0.2
 - primer-marketing-type@1.0.2
 - primer-marketing-utilities@1.0.2
 - primer-marketing@5.0.2
 - primer-navigation@1.0.6
 - primer-page-headers@1.0.2
 - primer-page-sections@1.0.2
 - primer-product@5.0.2
 - primer-support@4.0.7
 - primer-table-object@1.0.9
 - primer-tables@1.0.2
 - primer-tooltips@1.0.2
 - primer-truncate@1.0.2
 - primer-utilities@4.3.5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants