Tighten the vars-argument for the ESLint no-unused-vars rule

Please see https://eslint.org/docs/latest/rules/no-unused-vars#vars
This commit is contained in:
Jonas Jenwald 2022-12-03 11:54:56 +01:00
parent 67e1c37e0f
commit b659bacc43
4 changed files with 8 additions and 7 deletions

View File

@ -159,7 +159,7 @@
"no-undef-init": "error", "no-undef-init": "error",
"no-undef": ["error", { "typeof": true, }], "no-undef": ["error", { "typeof": true, }],
"no-unused-vars": ["error", { "no-unused-vars": ["error", {
"vars": "local", "vars": "all",
"args": "none", "args": "none",
}], }],
"no-use-before-define": ["error", { "no-use-before-define": ["error", {

View File

@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License. limitations under the License.
*/ */
/* import-globals-from pdfHandler.js */ /* import-globals-from pdfHandler.js */
/* exported saveReferer */
"use strict"; "use strict";
/** /**

View File

@ -14,8 +14,6 @@
*/ */
/* globals pdfjsLib, pdfjsViewer */ /* globals pdfjsLib, pdfjsViewer */
"use strict";
const { const {
AnnotationLayer, AnnotationLayer,
AnnotationMode, AnnotationMode,
@ -322,7 +320,6 @@ class Rasterize {
* @property {HTMLDivElement} end - Container for a completion message. * @property {HTMLDivElement} end - Container for a completion message.
*/ */
// eslint-disable-next-line no-unused-vars
class Driver { class Driver {
/** /**
* @param {DriverOptions} options * @param {DriverOptions} options
@ -962,3 +959,5 @@ class Driver {
return capability.promise; return capability.promise;
} }
} }
export { Driver };

View File

@ -20,7 +20,6 @@ limitations under the License.
<meta charset="utf-8"> <meta charset="utf-8">
<script src="../build/generic/build/pdf.js"></script> <script src="../build/generic/build/pdf.js"></script>
<script src="../build/components/pdf_viewer.js"></script> <script src="../build/components/pdf_viewer.js"></script>
<script src="driver.js"></script>
<link rel="resource" type="application/l10n" href="../build/generic/web/locale/locale.properties"> <link rel="resource" type="application/l10n" href="../build/generic/web/locale/locale.properties">
</head> </head>
@ -33,8 +32,10 @@ limitations under the License.
<pre id="output" style="max-height: 800px; overflow-y: scroll;"></pre> <pre id="output" style="max-height: 800px; overflow-y: scroll;"></pre>
<div id="end"></div> <div id="end"></div>
</body> </body>
<script> <script type="module">
var driver = new Driver({ import { Driver } from "./driver.js";
const driver = new Driver({
disableScrolling: document.getElementById('disableScrolling'), disableScrolling: document.getElementById('disableScrolling'),
inflight: document.getElementById('inflight'), inflight: document.getElementById('inflight'),
output: document.getElementById('output'), output: document.getElementById('output'),