Skip to content
This repository was archived by the owner on May 23, 2019. It is now read-only.

Make Browserify bundle work in ES3 environments#91

Merged
ericf merged 1 commit intoformatjs:masterfrom
ericf:browserify-es3
Jan 14, 2015
Merged

Make Browserify bundle work in ES3 environments#91
ericf merged 1 commit intoformatjs:masterfrom
ericf:browserify-es3

Conversation

@ericf
Copy link
Collaborator

@ericf ericf commented Jan 13, 2015

Fixes #90

@yahoocla
Copy link

CLA is valid!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I could conditionally use O.dP here if we prefer the default prop to be non-enum, etc. in ES5+ environments.

Copy link
Collaborator

Choose a reason for hiding this comment

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

na, we should be ok with enum.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We can also revisit this after looking into and understanding how 6to5/esnext deals (or will deal with) inter-module-system deps.

@caridy
Copy link
Collaborator

caridy commented Jan 13, 2015

LGTM

ericf added a commit that referenced this pull request Jan 14, 2015
Make Browserify bundle work in ES3 environments
@ericf ericf merged commit 1f33480 into formatjs:master Jan 14, 2015
@ericf ericf deleted the browserify-es3 branch January 14, 2015 03:48
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.

Internet Explorer 8 and Browserify

3 participants