Skip to content

Upgrade + Cleanup #327

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Upgrade + Cleanup #327

wants to merge 13 commits into from

Conversation

vish288
Copy link

@vish288 vish288 commented Aug 6, 2019

  1. Update dependencies to the latest version.
  2. Upgrade babel
  3. Upgrade eslint
  4. Upgrade Rollup
  5. Cleanup rollup config.
  6. Introduce Terser instead of Uglify
  7. Shrinkwrap to freeze dep.

mfeniseycopes and others added 13 commits May 19, 2017 08:38
@mfeniseycopes
Refactor index.js
08f504b
@mfeniseycopes
Fix noLogger ordering
0e9cbee
@mfeniseycopes
Have option checkers return boolean
a32491f
@mfeniseycopes
Add more specs/reorganize
e6c039a
@mfeniseycopes
noLogger -> hasLogger
ace19b8
Code Clean up
3d5cb30 
Indentation consistency
Removal of Lint issues.
Es6 literal usage.
Merge branch 'refactor'
7587ba6 
# Conflicts:
#	src/index.js
Fixes:
d936f20 
1. Update dependencies to the latest version.
2. Upgrade babel
3. Upgrade eslint
4. Upgrade Rollup
5. Cleanup rollup config.
6. Introduce Terser instead of Uglify
Added in shrinkwrap and lint issues clear.
d8f16ac
update dependencies and cleanup
dd3d15f
lint cleanup diff
5512b6d
merge conflicts resolution
4b470a2
merge conflicts resolution
303d145
@vish288 vish288 changed the title Cleanup Upgrade + Cleanup Aug 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@tomasreyes tomasreyes tomasreyes approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vish288 @tomasreyes @mfeniseycopes