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

Remove unused namespace declaration & expression #164

Merged
merged 2 commits into from Dec 18, 2017
Merged

Remove unused namespace declaration & expression #164

merged 2 commits into from Dec 18, 2017

Conversation

jsg2021
Copy link

@jsg2021 jsg2021 commented Aug 30, 2017

Its minor and optimized away in minification, but in a react-native environment these left-over bits cause problems.
The babelHelpers assignment somehow replaces the global one...which i think is a bug on their part, but I still think this is good to do.

Its minor and optimized away in minification, but in a react-native environment these left-over bits cause problems. 
The babelHelpers assignment somehow replaces the global one...which i think is a bug on their part, but I still think this is good to do.
@Johann-S
Copy link

Johann-S commented Aug 31, 2017

Related to : #165

@danez
Copy link
Collaborator

danez commented Dec 18, 2017

I guess this will not happen anymore with 4.0.0 as we now properly import the helpers.

//cc @Andarist

@danez danez added the 3.x label Dec 18, 2017
@danez danez changed the base branch from master to 3.x December 18, 2017 09:53
@danez
Copy link
Collaborator

danez commented Dec 18, 2017

Thanks I merged this into the 3.x branch. For 4.x this is not necessary anymore afaics

@danez danez merged commit 3ea75c4 into rollup:3.x Dec 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants