Skip to content

Commit

Permalink
Detect unused lint-disable directives
Browse files Browse the repository at this point in the history
  • Loading branch information
bobrippling committed Jun 25, 2024
1 parent 2c39e38 commit ccf6c2a
Show file tree
Hide file tree
Showing 11 changed files with 49 additions and 48 deletions.
1 change: 1 addition & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -245,4 +245,5 @@ module.exports = {
})),
],
ignorePatterns: findGeneratedJS(["apps/", "modules/"]),
reportUnusedDisableDirectives: true,
}
2 changes: 1 addition & 1 deletion apps/btadv/app.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
// @ts-expect-error helper
// eslint-disable-next-line @typescript-eslint/no-unused-vars

const __assign = Object.assign;

const Layout = require("Layout");
Expand Down
6 changes: 3 additions & 3 deletions apps/clkinfogps/geotools.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
*
*
* A module of Geo functions for use with gps fixes
*
* let geo = require("geotools");
Expand Down Expand Up @@ -71,7 +71,7 @@ OsGridRef.latLongToOsGrid = function(point) {
*
*/
function to_map_ref(digits, easting, northing) {
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`); // eslint-disable-line comma-spacing
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`);

let e = easting;
let n = northing;
Expand Down Expand Up @@ -108,7 +108,7 @@ function to_map_ref(digits, easting, northing) {
}

/**
*
*
* Module exports section, example code below
*
* let geo = require("geotools");
Expand Down
2 changes: 1 addition & 1 deletion apps/gipy/pkg/gps.d.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/* tslint:disable */
/* eslint-disable */

/**
* @param {Gps} gps
*/
Expand Down
2 changes: 1 addition & 1 deletion apps/gipy/pkg/gps_bg.wasm.d.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/* tslint:disable */
/* eslint-disable */

export const memory: WebAssembly.Memory;
export function __wbg_gps_free(a: number): void;
export function disable_elevation(a: number): void;
Expand Down
6 changes: 3 additions & 3 deletions apps/gpstouch/geotools.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
*
*
* A module of Geo functions for use with gps fixes
*
* let geo = require("geotools");
Expand Down Expand Up @@ -71,7 +71,7 @@ OsGridRef.latLongToOsGrid = function(point) {
*
*/
function to_map_ref(digits, easting, northing) {
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`); // eslint-disable-line comma-spacing
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`);

let e = easting;
let n = northing;
Expand Down Expand Up @@ -108,7 +108,7 @@ function to_map_ref(digits, easting, northing) {
}

/**
*
*
* Module exports section, example code below
*
* let geo = require("geotools");
Expand Down
2 changes: 1 addition & 1 deletion apps/kitchen/kitchen.app.js
Original file line number Diff line number Diff line change
Expand Up @@ -463,7 +463,7 @@ OsGridRef.latLongToOsGrid = function(point) {
*
*/
function to_map_ref(digits, easting, northing) {
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`); // eslint-disable-line comma-spacing
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`);

let e = easting;
let n = northing;
Expand Down
6 changes: 3 additions & 3 deletions apps/lint_exemptions.js
Original file line number Diff line number Diff line change
Expand Up @@ -926,7 +926,7 @@ module.exports = {
]
},
"apps/kitchen/kitchen.app.js": {
"hash": "1ef7b31e52110e34fb952d7ba0426c8bd9574e5f18be9fbc3b8ad1cc762dda21",
"hash": "cea726937a7179851091b0728d3ad1e773eac703a5bfdc28be6e2f247fdd44c9",
"rules": [
"no-undef"
]
Expand Down Expand Up @@ -1004,7 +1004,7 @@ module.exports = {
]
},
"apps/gpstouch/geotools.js": {
"hash": "5816fbb2dd630f574e5ee505e1b9ec6f80c3c53778b7a5520e5db28b91cdffc5",
"hash": "7e67733286f9d7708a54814f6f27d73ddffed2f433febc9604138f2f7a832cbf",
"rules": [
"no-undef"
]
Expand Down Expand Up @@ -1314,7 +1314,7 @@ module.exports = {
]
},
"apps/schoolCalendar/fullcalendar/main.js": {
"hash": "04dcd3cb3025c7aa67631d287b025a897b1cd984b8ea306abae2d722976fb7c5",
"hash": "8c417deb073328655117a93f045e77e9b808e84d584e648c6d7e360271ae8d07",
"rules": [
"no-undef",
"no-unused-vars",
Expand Down
18 changes: 9 additions & 9 deletions apps/schoolCalendar/fullcalendar/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ var FullCalendar = (function (exports) {
ContextType.Provider = function () {
var _this = this;
var isNew = !this.getChildContext;
var children = origProvider.apply(this, arguments); // eslint-disable-line prefer-rest-params
var children = origProvider.apply(this, arguments);
if (isNew) {
var subs_1 = [];
this.shouldComponentUpdate = function (_props) {
Expand Down Expand Up @@ -4688,14 +4688,14 @@ var FullCalendar = (function (exports) {
var wrappedSuccess = function () {
if (!isResolved) {
isResolved = true;
success.apply(this, arguments); // eslint-disable-line prefer-rest-params
success.apply(this, arguments);
}
};
var wrappedFailure = function () {
if (!isResolved) {
isResolved = true;
if (failure) {
failure.apply(this, arguments); // eslint-disable-line prefer-rest-params
failure.apply(this, arguments);
}
}
};
Expand Down Expand Up @@ -5008,7 +5008,7 @@ var FullCalendar = (function (exports) {
var createPortal = FullCalendarVDom.createPortal;
var flushToDom = FullCalendarVDom.flushToDom;
var unmountComponentAtNode = FullCalendarVDom.unmountComponentAtNode;
/* eslint-enable */


var ScrollResponder = /** @class */ (function () {
function ScrollResponder(execFunc, emitter, scrollTime, scrollTimeReset) {
Expand Down Expand Up @@ -5085,7 +5085,7 @@ var FullCalendar = (function (exports) {
}
PureComponent.prototype.shouldComponentUpdate = function (nextProps, nextState) {
if (this.debug) {
// eslint-disable-next-line no-console

console.log(getUnequalProps(nextProps, this.props), getUnequalProps(nextState, this.state));
}
return !compareObjs(this.props, nextProps, this.propEquality) ||
Expand Down Expand Up @@ -6613,7 +6613,7 @@ var FullCalendar = (function (exports) {
var endMarker = framingRange.end;
var instanceStarts = [];
while (dayMarker < endMarker) {
var instanceStart
var instanceStart
// if everyday, or this particular day-of-week
= void 0;
// if everyday, or this particular day-of-week
Expand Down Expand Up @@ -11731,7 +11731,7 @@ var FullCalendar = (function (exports) {
}
dragging.emitter.on('pointerdown', this.handlePointerDown);
dragging.emitter.on('dragstart', this.handleDragStart);
new ExternalElementDragging(dragging, settings.eventData); // eslint-disable-line no-new
new ExternalElementDragging(dragging, settings.eventData);
}
ExternalDraggable.prototype.destroy = function () {
this.dragging.destroy();
Expand Down Expand Up @@ -11833,7 +11833,7 @@ var FullCalendar = (function (exports) {
if (typeof settings.mirrorSelector === 'string') {
dragging.mirrorSelector = settings.mirrorSelector;
}
new ExternalElementDragging(dragging, settings.eventData); // eslint-disable-line no-new
new ExternalElementDragging(dragging, settings.eventData);
}
ThirdPartyDraggable.prototype.destroy = function () {
this.dragging.destroy();
Expand Down Expand Up @@ -13605,7 +13605,7 @@ var FullCalendar = (function (exports) {
if (!slatCoords) {
return null;
}
return segs.map(function (seg, i) { return (createElement(NowIndicatorRoot, { isAxis: false, date: date,
return segs.map(function (seg, i) { return (createElement(NowIndicatorRoot, { isAxis: false, date: date,
// key doesn't matter. will only ever be one
key: i }, function (rootElRef, classNames, innerElRef, innerContent) { return (createElement("div", { ref: rootElRef, className: ['fc-timegrid-now-indicator-line'].concat(classNames).join(' '), style: { top: slatCoords.computeDateTop(seg.start, date) } }, innerContent)); })); });
};
Expand Down
46 changes: 23 additions & 23 deletions apps/walkersclock/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
* - two function menus at present
* GPS Power = On/Off
* GPS Display = Grid | Speed Alt
* when the modeline in CYAN use button BTN1 to switch between options
* when the modeline in CYAN use button BTN1 to switch between options
* - display the current steps if one of the steps widgets is installed
* - ensures that BTN2 requires a 1.5 second press in order to switch to the launcher
* this is so you dont accidently switch out of the GPS/watch display with you coat sleeve
Expand Down Expand Up @@ -65,14 +65,14 @@ let last_fix = {
satellites: 0
};

function drawTime() {
function drawTime() {
var d = new Date();
var da = d.toString().split(" ");
var time = da[4].substr(0,5);

g.reset();
g.clearRect(0,Y_TIME, 239, Y_ACTIVITY - 1);

g.setColor(1,1,1); // white
g.setFontAlign(0, -1);

Expand All @@ -83,7 +83,7 @@ function drawTime() {
} else {
g.setFont("Vector", 80);
}

g.drawString(time, g.getWidth()/2, Y_TIME);
}

Expand All @@ -93,19 +93,19 @@ function drawActivity() {
clearActivityArea = true;

prevSteps = steps;

if (clearActivityArea) {
g.clearRect(0, Y_ACTIVITY, 239, Y_MODELINE - 1);
clearActivityArea = false;
}

if (!gpsPowerState) {
g.setColor(0,255,0); // green
g.setFont("Vector", 60);
g.drawString(getSteps(), g.getWidth()/2, Y_ACTIVITY);
return;
}

g.setFont("6x8", 3);
g.setColor(1,1,1);
g.setFontAlign(0, -1);
Expand All @@ -130,10 +130,10 @@ function drawActivity() {
let ref = to_map_ref(6, os.easting, os.northing);
let speed;
let activityStr = "";

if (age < 0) age = 0;
g.setFontVector(40);
g.setColor(0xFFC0);
g.setColor(0xFFC0);

switch(gpsDisplay) {
case GDISP_OS:
Expand All @@ -146,7 +146,7 @@ function drawActivity() {
case GDISP_SPEED:
speed = last_fix.speed;
speed = speed.toFixed(1);
activityStr = speed + "kph";
activityStr = speed + "kph";
break;
case GDISP_ALT:
activityStr = last_fix.alt + "m";
Expand All @@ -159,15 +159,15 @@ function drawActivity() {
g.clearRect(0, Y_ACTIVITY, 239, Y_MODELINE - 1);
g.drawString(activityStr, 120, Y_ACTIVITY);
g.setFont("6x8",2);
g.setColor(1,1,1);
g.setColor(1,1,1);
g.drawString(age, 120, Y_ACTIVITY + 46);
}
}

function onTick() {
if (!Bangle.isLCDOn())
return;

if (gpsPowerState) {
drawAll();
return;
Expand Down Expand Up @@ -226,7 +226,7 @@ function drawInfo() {
drawModeLine(str,col);
return;
}

switch(infoMode) {
case INFO_NONE:
col = 0x0000;
Expand All @@ -239,7 +239,7 @@ function drawInfo() {
default:
str = "Battery: " + E.getBattery() + "%";
}

drawModeLine(str,col);
}

Expand Down Expand Up @@ -283,7 +283,7 @@ function changeInfoMode() {
infoMode = INFO_NONE;
clearActivityArea = true;
return;

case FN_MODE_GDISP:
switch (gpsDisplay) {
case GDISP_OS:
Expand All @@ -304,7 +304,7 @@ function changeInfoMode() {
break;
}
}

switch(infoMode) {
case INFO_NONE:
if (stepsWidget() !== undefined)
Expand All @@ -319,7 +319,7 @@ function changeInfoMode() {
default:
infoMode = INFO_NONE;
}

clearActivityArea = true;
}

Expand Down Expand Up @@ -351,7 +351,7 @@ function changeFunctionMode() {
break;
}
}

infoMode = INFO_NONE; // function mode overrides info mode
}

Expand All @@ -374,7 +374,7 @@ function processFix(fix) {
gpsState = GPS_SATS;
clearActivityArea = true;
}

if (fix.fix) {
if (!last_fix.fix) {
if (!(require('Storage').readJSON('setting.json',1)||{}).quiet) {
Expand All @@ -401,7 +401,7 @@ function stepsWidget() {
}
return undefined;
}


/************* GPS / OSREF Code **************************/

Expand All @@ -413,10 +413,10 @@ function formatTime(now) {
function timeSince(t) {
var hms = t.split(":");
var now = new Date();

var sn = 3600*(now.getHours()) + 60*(now.getMinutes()) + 1*(now.getSeconds());
var st = 3600*(hms[0]) + 60*(hms[1]) + 1*(hms[2]);

return (sn - st);
}

Expand Down Expand Up @@ -483,7 +483,7 @@ OsGridRef.latLongToOsGrid = function(point) {
*
*/
function to_map_ref(digits, easting, northing) {
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`); // eslint-disable-line comma-spacing
if (![ 0,2,4,6,8,10,12,14,16 ].includes(Number(digits))) throw new RangeError(`invalid precision '${digits}'`);

let e = easting;
let n = northing;
Expand Down
Loading

0 comments on commit ccf6c2a

Please sign in to comment.