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

Upgrade stack #545

Merged
merged 2 commits into from Jul 5, 2018
Merged

Upgrade stack #545

merged 2 commits into from Jul 5, 2018

Conversation

TrySound
Copy link
Contributor

@TrySound TrySound commented Jul 4, 2018

I wanted to upgrade only babel but everything else was broken so I
upgraded babel, webpack, eslint and karma.

Disabled a lot of eslint rules to not change existing code. Maybe it's
better to migrate to prettier later.

This changes will unblock me from providing smaller output.

I wanted to upgrade only babel but everything else was broken so I
upgraded babel, webpack, eslint and karma.

Disabled a lot of eslint rules to not change existing code. Maybe it's
better to migrate to prettier later.

This changes will unblock me from providing smaller output.
@TrySound
Copy link
Contributor Author

TrySound commented Jul 4, 2018

/cc @chenglou

@TrySound
Copy link
Contributor Author

TrySound commented Jul 4, 2018

We need to rerun travis

@TrySound
Copy link
Contributor Author

TrySound commented Jul 5, 2018

Friendly ping @chenglou

@chenglou
Copy link
Owner

chenglou commented Jul 5, 2018

Thanks for the ping. To be clear, is there a user-facing change here? Or just for us devs?

@TrySound
Copy link
Contributor Author

TrySound commented Jul 5, 2018

@chenglou This one only for devs.

@TrySound
Copy link
Contributor Author

TrySound commented Jul 5, 2018

Bundle is slightly changed with new versions of webpack/babel but I don't think there's anything significant.

@TrySound
Copy link
Contributor Author

TrySound commented Jul 5, 2018

@chenglou Tests are passed.

@chenglou
Copy link
Owner

chenglou commented Jul 5, 2018

If there's no user-facing impact then sure. Thanks!

@chenglou chenglou merged commit d74daba into chenglou:master Jul 5, 2018
@TrySound TrySound deleted the upgrade-stack branch July 5, 2018 22:17
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

2 participants