Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code cleanup #4792

Merged
merged 7 commits into from Apr 3, 2019
Merged

Code cleanup #4792

merged 7 commits into from Apr 3, 2019

Conversation

fredj
Copy link
Member

@fredj fredj commented Apr 3, 2019

No description provided.

@gberaudo
Copy link
Member

gberaudo commented Apr 3, 2019

Looks like we have 123 import * in the code.
Some day these should be cleaned up to be made explicit and improve treeshaking:

gberaudo@wrk29:~/dev/ngeo (master $%)$ git grep 'import \* as' | wc -l
123

@fredj fredj merged commit 0aad938 into master Apr 3, 2019
@fredj fredj deleted the cleanup branch April 3, 2019 10:03
@sbrunner
Copy link
Member

sbrunner commented Apr 3, 2019

Thanks :-)

@sbrunner sbrunner added this to the 2.4 milestone Apr 8, 2019
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.

None yet

3 participants