Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Document browser field #18382

Merged
merged 2 commits into from Mar 8, 2018
Merged

Document browser field #18382

merged 2 commits into from Mar 8, 2018

Conversation

mxstbr
Copy link
Contributor

@mxstbr mxstbr commented Sep 2, 2017

Based on https://github.com/defunctzombie/package-browser-field-spec and https://twitter.com/seldo/status/903997474645434368 this adds a very short note about the browser field to the package.json docs.

/cc @seldo @mjackson @wycats

@mxstbr
Copy link
Contributor Author

mxstbr commented Sep 2, 2017

Note that this was shittly worded, I had to get out at my stop of the train, any suggestions very welcome!

Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

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

I think this is ok to include in here. We don't do anything with it right now, but I think the field's become widespread enough to warrant this.

@zkat zkat changed the base branch from master to release-next March 6, 2018 23:06
@zkat zkat merged this pull request into npm:release-next Mar 8, 2018
zkat pushed a commit that referenced this pull request Mar 8, 2018
Based on https://github.com/defunctzombie/package-browser-field-spec and https://twitter.com/seldo/status/903997474645434368 this adds a very short note about the `browser` field to the package.json docs.

PR-URL: #18382
Credit: @mxstbr
Reviewed-By: @zkat
@mxstbr mxstbr deleted the patch-1 branch March 9, 2018 08:02
@mxstbr
Copy link
Contributor Author

mxstbr commented Mar 9, 2018

💯

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

Successfully merging this pull request may close these issues.

None yet

2 participants