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

Eslint shut #591

Open
wants to merge 2 commits into
base: v2
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
6 changes: 3 additions & 3 deletions injectors/elevate.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ const { spawnSync } = require('child_process');

// It seems `sudo npm ...` no longer give the script sudo perms in npm v7, so here we are.
if (process.platform === 'linux' && process.getuid() !== 0) {
// todo: decide whether this is a viable solution for the long term. people may use doas or other tools for elevating themselves.
spawnSync('sudo', process.argv, { stdio: 'inherit' })
process.exit(0)
// @todo: decide whether this is a viable solution for the long term. people may use doas or other tools for elevating themselves.
spawnSync('sudo', process.argv, { stdio: 'inherit' });
process.exit(0);
}
4 changes: 2 additions & 2 deletions injectors/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ require('./env_check')(); // Perform checks
require('../polyfills'); // And then do stuff

const { join } = require('path');
const { readFile, writeFile } = require('fs').promises;
const { writeFile } = require('fs').promises;
const { BasicMessages } = require('./log');
const main = require('./main.js');

Expand Down Expand Up @@ -50,7 +50,7 @@ try {
}
})().catch(e => {
if (e.code === 'EACCES') {
// todo: this was linux only (?) so I assume this is now safe to delete
// @todo: this was linux only (?) so I assume this is now safe to delete
console.log(process.argv[2] === 'inject' ? BasicMessages.PLUG_FAILED : BasicMessages.UNPLUG_FAILED, '\n');
console.log('Powercord wasn\'t able to inject itself due to missing permissions.', '\n');
console.log('Try again with elevated permissions.');
Expand Down
1 change: 1 addition & 0 deletions src/Powercord/plugins/pc-stafftags
Submodule pc-stafftags added at 85f93e
2 changes: 1 addition & 1 deletion src/patcher.js
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ electron.app.name = discordPackage.name;
* @see https://github.com/electron/electron/issues/19468
*/
if (process.platform === 'win32') {
// todo: define if this is still necessary
// @todo: define if this is still necessary
setImmediate(() => { // WTF: the app name doesn't get set instantly?
const devToolsExtensions = join(electron.app.getPath('userData'), 'DevTools Extensions');

Expand Down
2 changes: 1 addition & 1 deletion src/preload.js
Original file line number Diff line number Diff line change
Expand Up @@ -300,7 +300,7 @@ if (preload) {
setTimeout(() => DiscordNative.window.setDevtoolsCallbacks(null, null), 5e3);

/*
todo: rewrite these
@todo: rewrite these
// Debug logging
let debugLogs = false;
try {
Expand Down