Browse Source

Remove eslint-disable comments (#4681)

Do not reject console.error and console.warn with ESLint rules.
pull/4/head
Yamagishi Kazutoshi 6 years ago
committed by Eugen Rochko
parent
commit
fbe1115114
3 changed files with 2 additions and 5 deletions
  1. +1
    -0
      .eslintrc.yml
  2. +1
    -1
      app/javascript/mastodon/features/ui/index.js
  3. +0
    -4
      app/javascript/mastodon/web_push_subscription.js

+ 1
- 0
.eslintrc.yml View File

@ -49,6 +49,7 @@ rules:
- warn
- allow:
- error
- warn
no-fallthrough: error
no-irregular-whitespace: error
no-mixed-spaces-and-tabs: warn

+ 1
- 1
app/javascript/mastodon/features/ui/index.js View File

@ -135,7 +135,7 @@ export default class UI extends React.PureComponent {
if (data.type === 'navigate') {
this.context.router.history.push(data.path);
} else {
console.warn('Unknown message type:', data.type); // eslint-disable-line no-console
console.warn('Unknown message type:', data.type);
}
}

+ 0
- 4
app/javascript/mastodon/web_push_subscription.js View File

@ -48,7 +48,6 @@ export function register () {
if (supportsPushNotifications) {
if (!getApplicationServerKey()) {
// eslint-disable-next-line no-console
console.error('The VAPID public key is not set. You will not be able to receive Web Push Notifications.');
return;
}
@ -84,10 +83,8 @@ export function register () {
})
.catch(error => {
if (error.code === 20 && error.name === 'AbortError') {
// eslint-disable-next-line no-console
console.warn('Your browser supports Web Push Notifications, but does not seem to implement the VAPID protocol.');
} else if (error.code === 5 && error.name === 'InvalidCharacterError') {
// eslint-disable-next-line no-console
console.error('The VAPID public key seems to be invalid:', getApplicationServerKey());
}
@ -103,7 +100,6 @@ export function register () {
}
});
} else {
// eslint-disable-next-line no-console
console.warn('Your browser does not support Web Push Notifications.');
}
}

Loading…
Cancel
Save