Skip to content

Update dev deps + fix linting issues#382

Merged
bajtos merged 4 commits into
masterfrom
update-dev-deps
Jul 8, 2019
Merged

Update dev deps + fix linting issues#382
bajtos merged 4 commits into
masterfrom
update-dev-deps

Conversation

@bajtos

@bajtos bajtos commented Jul 4, 2019

Copy link
Copy Markdown
Member

I noticed that this connector is failing CI builds triggered by pull request in loopback-datasource-juggler. In this pull request, I am upgrading all dev-dependencies to the latest versions and fixing eslint violations.

  • a266900 updates sinon, mocha, should
  • d57001f updates eslint
  • 97b3343 contains changes created by eslint . --fix, I guess there is not much to review there
  • be7e95c is fixing the remaining places that eslint could not fix automatically

Related issues

  • n/a

Checklist

  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide

bajtos added 4 commits July 4, 2019 14:02
- mocha to ^6.1.4
- should to ^13.2.3
- sinon to ^7.3.2

Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
@bajtos
bajtos requested review from elv1s, jannyHou and zbarbuto July 4, 2019 12:16
@bajtos
bajtos requested review from b-admike and dhmlau as code owners July 4, 2019 12:16
@bajtos bajtos self-assigned this Jul 4, 2019

@jannyHou jannyHou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@bajtos
bajtos merged commit 91ffb7b into master Jul 8, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the update-dev-deps branch July 8, 2019 08:43
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.

4 participants