Skip to content

Expose lib from browser field#7

Merged
3rd-Eden merged 3 commits intogodaddy:masterfrom
kinetifex:browser-field
Oct 12, 2018
Merged

Expose lib from browser field#7
3rd-Eden merged 3 commits intogodaddy:masterfrom
kinetifex:browser-field

Conversation

@kinetifex
Copy link
Copy Markdown

No description provided.

Comment thread package.json
"main": "./lib",
"browser": "./index",
"browser": "./lib",
"module": "./index",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We probably want to ensure that react-native still receives the uncompiled library, so you will end up with the following keys;

  "main": "./lib",
  "browser": "./lib",
  "module": "./index",
  "react-native": "./index",

Copy link
Copy Markdown
Contributor

@msluther msluther left a comment

Choose a reason for hiding this comment

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

Once @3rd-Eden 's comments are addressed.

@3rd-Eden 3rd-Eden merged commit 5b2fc10 into godaddy:master Oct 12, 2018
@3rd-Eden
Copy link
Copy Markdown
Contributor

Published as 4.0.0

@kinetifex kinetifex deleted the browser-field branch October 12, 2018 16:17
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.

5 participants