Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Update eslint and fix formatting #92

Merged
merged 1 commit into from
May 29, 2018
Merged

Conversation

fhinkel
Copy link
Contributor

@fhinkel fhinkel commented May 28, 2018

Fix eslint error in startup-script sample.

Fixes https://circleci.com/gh/googleapis/nodejs-compute/2294.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 28, 2018
@ghost ghost assigned fhinkel May 28, 2018
@fhinkel fhinkel force-pushed the update-eslint branch 2 times, most recently from c3585e6 to 9e8b852 Compare May 28, 2018 15:30
@codecov
Copy link

codecov bot commented May 28, 2018

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #92   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          19     19           
  Lines        1583   1583           
=====================================
  Hits         1583   1583

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 557ae71...d00ad82. Read the comment docs.

@fhinkel
Copy link
Contributor Author

fhinkel commented May 28, 2018

Update package-json.lock as the eslint rules apparently have changed.

"Error: Delete ⏎····" on the new version versus "Error: Insert ⏎····" on the old version on the same line.

@fhinkel fhinkel requested a review from alexander-fenster May 28, 2018 15:55
@alexander-fenster alexander-fenster merged commit 008dee3 into master May 29, 2018
@ghost ghost removed the cla: yes This human has signed the Contributor License Agreement. label May 29, 2018
@alexander-fenster alexander-fenster deleted the update-eslint branch May 29, 2018 19:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants