Skip to content
This repository has been archived by the owner on Jul 19, 2021. It is now read-only.

Update normalize scss to v7 #194

Merged
merged 1 commit into from
Jul 11, 2017
Merged

Update normalize scss to v7 #194

merged 1 commit into from
Jul 11, 2017

Conversation

t-kelly
Copy link
Contributor

@t-kelly t-kelly commented Jul 11, 2017

What are you trying to accomplish with this PR?

Fixes #121

For maintainers:

  • I have 🎩'd these changes.
  • I have bumped the package.json version in a separate PR, if applicable.

@bertiful
Copy link
Contributor

Doesn't seem like it, but this wouldn't have any unforeseen repercussions with any other Slate styling would it?

@@ -1,163 +1,171 @@
/*! normalize.css v3.0.3 | MIT License | github.com/necolas/normalize.css */
/*! normalize.css v6.0.0 | MIT License | github.com/necolas/normalize.css */
Copy link
Contributor

@huguestennier huguestennier Jul 11, 2017

Choose a reason for hiding this comment

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

v7.0.0 is out - any reason to use v6.0.0 instead?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, seems to just be some small tweaks: necolas/normalize.css@b4a8fda

Should be fine to upgrade to v7.0.0.

@t-kelly
Copy link
Contributor Author

t-kelly commented Jul 11, 2017

@huguestennier updated

@t-kelly t-kelly merged commit e4aee69 into 0.11.0 Jul 11, 2017
@t-kelly t-kelly changed the title Update normalize scss to v6 Update normalize scss to v7 Jul 11, 2017
@t-kelly t-kelly deleted the normalize-css branch July 11, 2017 17:45
@lock
Copy link

lock bot commented Oct 26, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants