X Tutup
The Wayback Machine - https://web.archive.org/web/20200916183700/https://github.com/nodegit/nodegit/pull/1791
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

migrate jshint to eslint #1791

Open
wants to merge 2 commits into
base: master
from

Conversation

@ChristianMurphy
Copy link

ChristianMurphy commented Sep 3, 2020

configuration migration automated using https://github.com/brenolf/polyjuice.
find and replace used to switch jshint ignore comments for eslint disable comments.

alternative to #1338


there are a few unused vars and uses of proto, which jshint missed that eslint catches, that are being flagged.

nodegit/lib/nodegit.js
  1081:1  error  The '__proto__' property is deprecated  no-proto

nodegit/lib/repository.js
  1466:20  error  'promiseChain' is assigned a value but never used  no-unused-vars

nodegit/test/tests/commit.js
  1045:5  error  'signature' is assigned a value but never used  no-unused-vars

nodegit/test/tests/merge.js
   179:11  error  'ourBranch' is assigned a value but never used    no-unused-vars
   213:9   error  'theirCommit' is assigned a value but never used  no-unused-vars
   295:11  error  'ourBranch' is assigned a value but never used    no-unused-vars
   329:9   error  'theirCommit' is assigned a value but never used  no-unused-vars
   454:9   error  'ourCommit' is assigned a value but never used    no-unused-vars
   485:9   error  'theirCommit' is assigned a value but never used  no-unused-vars
   604:11  error  'ourCommit' is assigned a value but never used    no-unused-vars
   635:11  error  'theirCommit' is assigned a value but never used  no-unused-vars
   763:11  error  'ourCommit' is assigned a value but never used    no-unused-vars
   794:11  error  'theirCommit' is assigned a value but never used  no-unused-vars
   881:11  error  'ourBranch' is assigned a value but never used    no-unused-vars
   915:9   error  'theirCommit' is assigned a value but never used  no-unused-vars
  1037:9   error  'ourCommit' is assigned a value but never used    no-unused-vars
  1068:9   error  'theirCommit' is assigned a value but never used  no-unused-vars
  1184:9   error  'ourCommit' is assigned a value but never used    no-unused-vars
  1215:9   error  'theirCommit' is assigned a value but never used  no-unused-vars

nodegit/test/tests/revwalk.js
  114:11  error  'storedCommits' is assigned a value but never used  no-unused-vars

nodegit/test/tests/signature.js
  103:5  error  'time' is assigned a value but never used  no-unused-vars

nodegit/test/tests/stash.js
  157:11  error  'oldContent' is assigned a value but never used  no-unused-vars
  189:11  error  'oldContent' is assigned a value but never used  no-unused-vars

✖ 23 problems (23 errors, 0 warnings)

these could either be ignored or the tests refactored to use the variable.

configuration migration automated using https://github.com/brenolf/polyjuice.
find and replace used to switch jshint ignore comments for eslint disable comments.
@implausible
Copy link
Member

implausible commented Sep 4, 2020

I actually have a preference for how to go about this one. I was working on bringing this into the library here:
https://github.com/implausible/nodegit/tree/convert-to-es6. Need to rebase that on top of master and address any of the major changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.
X Tutup