Skip to content

/ ghostery-extension

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

Jump to bottom

Tool/unused i18n token finder #457

Closed
wants to merge 8 commits into from
+133 −7
Closed

Tool/unused i18n token finder #457

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
015c043
Implement core of a tool that identifies unused i18n tokens
wlycdgr Oct 2, 2019
9064846
Clean up and document unused i18n token finder script
wlycdgr Oct 3, 2019
11ec40e
Merge branch 'develop' into tool/unused-i18n-token-finder
wlycdgr Oct 9, 2019
eb21b9f
Remove 'possibly' from unused i18n token finder script name
wlycdgr Oct 10, 2019
b20fd10
Create i18n_results directory if needed when writing unused i18n toke…
wlycdgr Oct 10, 2019
80e0adb
Remove a redundant return statement in unused i18n token finder
wlycdgr Oct 10, 2019
bab629a
Add execution time logging to unused i18n token finder script
wlycdgr Oct 10, 2019
1ff575f
Fix path bugs in the original i18n-checker script
wlycdgr Oct 10, 2019
File filter
Filter file types
Clear filters
Jump to
Jump to file
Failed to load files.

Always

Just for now

Prev
Loading status checks…
Fix path bugs in the original i18n-checker script
  • Loading branch information
@wlycdgr
wlycdgr committed Oct 10, 2019
commit 1ff575f485f3f3b61a04c7d34c1dcb1a06415415
14 tools/i18n-checker.js
View file
Open in desktop
@@ -24,7 +24,7 @@ const oboe = require('oboe');
const LOCALES_FOLDER = './_locales';
const GATHER_FILE_PATHS_EXCEPTIONS = ['.DS_Store'];
const LANG_FILES_COUNT = 14;
const DEFAULT_LOCALE_PATH = '../_locales/en/messages.json';
const DEFAULT_LOCALE_PATH = './_locales/en/messages.json';
const DUPLICATE_TOKENS_FILE = './tools/i18n_results/duplicate_tokens.txt';
const MISSING_TOKENS_FILE = './tools/i18n_results/missing_tokens.txt';
const EXTRA_TOKENS_FILE = './tools/i18n_results/extra_tokens.txt';
@@ -175,7 +175,7 @@ function findMissingKeys(paths) {
let hasMissingKeys = false;
const missingKeys = {};
paths.forEach((path) => {
const localeJson = jsonfile.readFileSync(`.${path}`);
const localeJson = jsonfile.readFileSync(`${path}`);
const locale = path.match(/_locales\/(.*)\/messages.json/)[1];
missingKeys[locale] = [];
Object.keys(defaultLocaleJson).forEach((key) => {
@@ -210,7 +210,7 @@ function findExtraKeys(paths) {
let hasExtraKeys = false;
const extraKeys = {};
paths.forEach((path) => {
const localeJson = jsonfile.readFileSync(`.${path}`);
const localeJson = jsonfile.readFileSync(`${path}`);
const locale = path.match(/_locales\/(.*)\/messages.json/)[1];
extraKeys[locale] = [];
Object.keys(localeJson).forEach((key) => {
@@ -243,7 +243,7 @@ function findMalformedKeys(paths) {
let hasMalformedKeys = false;
const malformedKeys = {};
paths.forEach((path) => {
const localeJson = jsonfile.readFileSync(`.${path}`);
const localeJson = jsonfile.readFileSync(`${path}`);
const locale = path.match(/_locales\/(.*)\/messages.json/)[1];
malformedKeys[locale] = [];
Object.keys(localeJson).forEach((key) => {
@@ -278,7 +278,7 @@ function findMissingPlaceholders(paths) {
let hasMissingPlaceholders = false;
const missingPlaceholders = {};
paths.forEach((path) => {
const localeJson = jsonfile.readFileSync(`.${path}`);
const localeJson = jsonfile.readFileSync(`${path}`);
const locale = path.match(/_locales\/(.*)\/messages.json/)[1];
missingPlaceholders[locale] = [];
Object.keys(defaultLocaleJson).forEach((key) => {
@@ -322,7 +322,7 @@ function findExtraPlaceholders(paths) {
let hasExtraPlaceholders = false;
const extraPlaceholders = {};
paths.forEach((path) => {
const localeJson = jsonfile.readFileSync(`.${path}`);
const localeJson = jsonfile.readFileSync(`${path}`);
const locale = path.match(/_locales\/(.*)\/messages.json/)[1];
extraPlaceholders[locale] = [];
Object.keys(localeJson).forEach((key) => {
@@ -364,7 +364,7 @@ function findMalformedPlaceholders(paths) {
let hasMalformedPlaceholders = false;
const malformedPlaceholders = [];
paths.forEach((path) => {
const localeJson = jsonfile.readFileSync(`.${path}`);
const localeJson = jsonfile.readFileSync(`${path}`);
const locale = path.match(/_locales\/(.*)\/messages.json/)[1];
malformedPlaceholders[locale] = [];
Object.keys(localeJson).forEach((key) => {
ProTip! Use n and p to navigate between commits in a pull request.