search

internetarchive / bookreader

The Internet Archive BookReader
https://openlibrary.org/dev/docs/bookreader
GNU Affero General Public License v3.0
996 stars 418 forks source link

Add eslint trailing comma rule #1347

Closed cdrini closed 1 month ago

cdrini commented 1 month ago

CC @bfalling . Response to https://github.com/internetarchive/bookreader/pull/1345#discussion_r1785498293

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 46.15385% with 7 lines in your changes missing coverage. Please review.

Project coverage is 69.25%. Comparing base (62add6c) to head (174edf2). Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/BookReader.js 40.00% 2 Missing and 1 partial :warning:
src/BookNavigator/book-navigator.js 0.00% 1 Missing :warning:
src/BookReader/Toolbar/Toolbar.js 0.00% 1 Missing :warning:
src/plugins/plugin.text_selection.js 0.00% 1 Missing :warning:
src/plugins/tts/AbstractTTSEngine.js 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1347 +/- ## ======================================= Coverage 69.25% 69.25% ======================================= Files 59 59 Lines 5090 5090 Branches 1072 1072 ======================================= Hits 3525 3525 Misses 1538 1538 Partials 27 27 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cdrini commented 1 month ago

Auto-merging, small lint change implied by previous PR comment.