-
Notifications
You must be signed in to change notification settings - Fork 30
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
chore(condo): DOMA-10737 improve lint-i18n-translations.js #5530
base: main
Are you sure you want to change the base?
Conversation
@@ -3,55 +3,59 @@ const fetch = require('node-fetch') | |||
|
|||
const { getAppServerUrl } = require('@open-condo/cli') | |||
|
|||
program.option('-f, --filter <names...>', 'Filters apps by name') | |||
program.option('-s, --sleep <number>', 'Sleep interval between polling in ms', /\d{1,12}/, '3000') | |||
program.option('-t, --timeout <number>', 'Max polling time in ms', /\d{1,12}/, '120000') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RegExp is deprecated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like we don't use a "RESET_PASSWORD" message type, so you should remove translations for that type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I want to add this check to the linter
Quality Gate passedIssues Measures |
No description provided.