add new build:ci and fix l10ncheck bug (#6284)

microsoft · Mar 6, 2021 · ec7f61c · ec7f61c
1 parent 2b8848b
commit ec7f61c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/Composer/package.json b/Composer/package.json
@@ -42,6 +42,7 @@
   "scripts": {
     "build": "node scripts/begin.js && yarn build:prod && yarn l10n",
     "build:prod": "yarn build:dev && yarn build:client && yarn build:server && yarn build:electron",
+    "build:ci": "yarn l10n:check && yarn build:dev",
     "build:dev": "yarn build:libs && yarn build:extensions",
     "build:electron": "yarn workspace @bfc/electron-server build && yarn workspace @bfc/electron-server l10n",
     "build:libs": "wsrun -ltm -x @bfc/electron-server -x @bfc/client -x @bfc/server -p @botframework-composer/* -p @bfc/* -c build",

diff --git a/Composer/scripts/l10nCheck.js b/Composer/scripts/l10nCheck.js
@@ -18,6 +18,7 @@ for (;;) {
   const json = JSON.parse(data);
   for (const [key, val] of Object.entries(json)) {
     const msg = val.message;
+
     const src = `${dirent.name} at ${key}:`;
     if (/\{.*plural.*}/.exec(msg)) {
       // if we're in a "plural" rules section...
@@ -30,7 +31,7 @@ for (;;) {
         errorCount += 1;
       }
     }
-    if (/(\p{L}|\s)'(\p{L})/.exec(msg)) {
+    if (/(\p{L}|\s|\w)'(\p{L}|\w)/.exec(msg)) {
       console.log(src, `single quote between letters`);
       errorCount += 1;
     }