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

Update free-style to version 3.1.0 and other deps updates #350

Merged
merged 4 commits into from
Mar 15, 2020

Conversation

notoriousb1t
Copy link
Contributor

Closes #346

greenkeeper bot and others added 4 commits March 5, 2020 01:54
1. Upgraded to freestyle 3.1.0
   a) The $displayName property was added in 3.0.0 and is effectively the
      $debugName we were using. Because freestyle does not have the debug
      parameter anymore, the code around $debugName has been simplified to
      passthrough to the $displayName property.
   b) The IS_UNIQUE property was changed to be $unique. This is the same
      property name that is used in TypeStyle, so no special handling is
      needed. All the special handling has been removed.
   c) Due to these changes, we should treat this as a major version.
2. Upgraded webpack
   a) Added mode.
   b) Replaced uglify with optimization property. Uglify plugin is no
      longer supported
3. Renamed internal formatting functions to convertToKeyframes and
   convertToStyles. Also simplified the types used.
@notoriousb1t notoriousb1t changed the title fix(package): update free-style to version 3.1.0 Update free-style to version 3.1.0 and other deps updates Mar 14, 2020
@notoriousb1t
Copy link
Contributor Author

@basarat, could you spot check this for me? I haven't made significant changes to this codebase in a while and I haven't had any opportunity to do React development since I joined Google.

I actually meant to just hit some merge buttons to clear my inbox, but the free-style api changed and many of our deps were pretty out of date. It looks like this also closes our other PR for $debugName since free-style depends on node environment now and doesn't have a parameter for this. This fixes that request.

Let me know what you think. If this looks good, I'll up the major version and see what changes need to be made to the docs site.

@basarat
Copy link
Contributor

basarat commented Mar 14, 2020

Let me know what you think. If this looks good, I'll up the major version and see what changes need to be made to the docs site.

Looks good 👍

@notoriousb1t notoriousb1t merged commit 5e73b7e into master Mar 15, 2020
@greenkeeper greenkeeper bot deleted the greenkeeper/free-style-3.1.0 branch March 15, 2020 00:21
@basarat
Copy link
Contributor

basarat commented Apr 1, 2020

@notoriousb1t from commit : 5e73b7e

Due to these changes, we should treat this as a major version.

I disagree. I don't see any breaking changes for our consumers : our $debugName still works, our $unique still works.

If you are okay I'll cut a minor release to include this along with other changes (that are also minor) 👍

@notoriousb1t
Copy link
Contributor Author

Perhaps I was being overly cautious here. SGTM 👍

@basarat
Copy link
Contributor

basarat commented Apr 1, 2020

Released 2.1.0 🌹

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.

2 participants