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

fix(babelrc): add a key for each env development, production, es #547

Merged
merged 3 commits into from
Nov 2, 2017

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Nov 1, 2017

Summary

Fix #499

Since babel@7 is not yet release, I just updated the .babelrc to correctly output ES modules.
The merge of the presets with the env seems to not work, so I duplicated each configuration. As soon as babel@7 is out we should migrate to .babelrc.js as @Haroenv suggest.

@algobot
Copy link
Contributor

algobot commented Nov 1, 2017

Deploy preview ready!

Built with commit 23b4928

https://deploy-preview-547--react-instantsearch.netlify.com

@Haroenv
Copy link
Contributor

Haroenv commented Nov 1, 2017

Good idea, it breaks the tests though, I know that Jest can only have the test env, and not the fallback development, might be related

@samouss
Copy link
Collaborator Author

samouss commented Nov 1, 2017

@Haroenv Yep, you are right I forgot that... Should be good now.

Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

correctly has es modules in the es folder; tests pass; LGTM!

@samouss samouss merged commit fa9528d into master Nov 2, 2017
@samouss samouss deleted the fix/babel-env branch November 2, 2017 17:12
samouss added a commit that referenced this pull request Nov 27, 2017
<a name="4.3.0-beta.0"></a>
# [4.3.0-beta.0](v4.2.0...v4.3.0-beta.0) (2017-11-27)

### Bug Fixes

* **babelrc:** add a key for each env development, production, es ([#547](#547)) ([fa9528d](fa9528d))
* **localizecount:** allow localized string for count in MenuSelect ([#657](#657)) ([67ebd34](67ebd34))
* **react-router-example:** Properly update search query when using browser navigation ([#604](#604)) ([9ee6600](9ee6600))

### Features

* **refreshcache:** add prop refresh to InstantSearch instance ([#619](#619)) ([19f6de0](19f6de0))
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