Skip to content

Clean up eslint comments #2197

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.

Sign up for GitHub

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

Merged
marcustyphoon merged 1 commit into from
Sep 5, 2025
Merged

Clean up eslint comments #2197

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions Extensions/one_click_postage.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1108,8 +1108,6 @@ XKit.extensions.one_click_postage = new Object({
}
}
}, 700);

/* eslint-enable no-undef */
},

/**
Expand Down
2 changes: 1 addition & 1 deletion Extensions/one_click_reply.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ XKit.extensions.one_click_reply = new Object({
});
}
return R('\n</p>$', '</p>');
/* eslint-disable id-length */
/* eslint-enable id-length */
},

quick_reply_open: function(sentence, default_tags, avatar, username) {
Expand Down
6 changes: 4 additions & 2 deletions Extensions/outbox.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,10 @@ XKit.extensions.outbox = new Object({
}
const date = new Date();
date.setTime(time);

/* eslint-disable no-sparse-arrays */
if (answer.length) {
text +=
// eslint-disable-next-line no-sparse-arrays
[
date.toLocaleString(),
`Private answer from ${to}:`,
Expand All @@ -99,7 +100,6 @@ XKit.extensions.outbox = new Object({
].join('\n');
} else {
text +=
// eslint-disable-next-line no-sparse-arrays
[
date.toLocaleString(),
`You asked ${to}:`,
Expand All @@ -109,6 +109,8 @@ XKit.extensions.outbox = new Object({
` - ${username}`,
].join('\n');
}
/* eslint-enable no-sparse-arrays */

text += '\n\n\n\n\n';
});
});
Expand Down
2 changes: 1 addition & 1 deletion Extensions/servant.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1254,7 +1254,7 @@ XKit.extensions.servant = new Object({
for (var i = 0; i < colorArray.length; i++) {
colorArray[i] = colorArray[i].replace("_", "");
}
var uniques = [...new Set(colorArray)]; //eslint-disable-line no-undef
var uniques = [...new Set(colorArray)];
var backgroundString = "";
for (var j = 0; j < uniques.length; j++) {
backgroundString += `, #${uniques[j]} ${(100 / uniques.length) * j}%, #${uniques[j]} ${(100 / uniques.length) * (j + 1)}%`;
Expand Down
2 changes: 1 addition & 1 deletion Extensions/xkit_patches.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3115,7 +3115,7 @@ XKit.extensions.xkit_patches = new Object({
}, 100);
$("#xkit_notification_" + m_notification_id).click(function() {
// TODO: fix this erroneous comparison
// eslint-disable-next-line valid-typeof
// eslint-disable-next-line
if (typeof callback !== undefined) {
try {
callback();
Expand Down
2 changes: 1 addition & 1 deletion xkit.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -821,7 +821,7 @@ var xkit_global_start = Date.now(); // log start timestamp
}, 100);
$("#xkit_notification_" + m_notification_id).click(function() {
// TODO: fix this erroneous comparison
// eslint-disable-next-line valid-typeof
// eslint-disable-next-line
if (typeof callback !== undefined) {
try {
callback();
Expand Down