Update prettier dev-dependency to v2.5.1 in Communication (Azure#19489)

* Format communication-chat

* Format communcation-common

* Format communication-identity

* Format communication-network-traversal

* Format communication-phone-numbers

* Format communication-short-codes

* Format communication-sms

* Rush update

* rush update and format on communication-sms

Author: JonathanCrd

common/config/rush/pnpm-lock.yaml

@@ -5,10 +5,10 @@ const {
   jsonRecordingFilterFunction,
   isPlaybackMode,
   isSoftRecordMode,
-  isRecordMode
+  isRecordMode,
 } = require("@azure-tools/test-recorder");
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     // base path that will be used to resolve all patterns (eg. files, exclude)
     basePath: "./",
@@ -29,7 +29,7 @@ module.exports = function(config) {
       "karma-sourcemap-loader",
       "karma-junit-reporter",
       "karma-json-to-file-reporter",
-      "karma-json-preprocessor"
+      "karma-json-preprocessor",
     ],
 
     // list of files / patterns to load in the browser
@@ -44,7 +44,7 @@ module.exports = function(config) {
     // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
     preprocessors: {
       "**/*.js": ["sourcemap", "env"],
-      "recordings/browsers/**/*.json": ["json"]
+      "recordings/browsers/**/*.json": ["json"],
       // IMPORTANT: COMMENT following line if you want to debug in your browsers!!
       // Preprocess source file to calculate code coverage, however this will make source file unreadable
       //"dist-test/index.browser.js": ["coverage"]
@@ -67,8 +67,8 @@ module.exports = function(config) {
         { type: "json", subdir: ".", file: "coverage.json" },
         { type: "lcovonly", subdir: ".", file: "lcov.info" },
         { type: "html", subdir: "html" },
-        { type: "cobertura", subdir: ".", file: "cobertura-coverage.xml" }
-      ]
+        { type: "cobertura", subdir: ".", file: "cobertura-coverage.xml" },
+      ],
     },
 
     junitReporter: {
@@ -78,12 +78,12 @@ module.exports = function(config) {
       useBrowserName: false, // add browser name to report and classes names
       nameFormatter: undefined, // function (browser, result) to customize the name attribute in xml testcase element
       classNameFormatter: undefined, // function (browser, result) to customize the classname attribute in xml testcase element
-      properties: {} // key value pair of properties to add to the <properties> section of the report
+      properties: {}, // key value pair of properties to add to the <properties> section of the report
     },
 
     jsonToFileReporter: {
       filter: jsonRecordingFilterFunction,
-      outputPath: "."
+      outputPath: ".",
     },
 
     // web server port
@@ -107,8 +107,8 @@ module.exports = function(config) {
     customLaunchers: {
       HeadlessChrome: {
         base: "ChromeHeadless",
-        flags: ["--no-sandbox"]
-      }
+        flags: ["--no-sandbox"],
+      },
     },
 
     // Continuous Integration mode
@@ -123,15 +123,15 @@ module.exports = function(config) {
     browserDisconnectTimeout: 10000,
     browserDisconnectTolerance: 3,
     browserConsoleLogOptions: {
-      terminal: !isRecordMode()
+      terminal: !isRecordMode(),
     },
 
     client: {
       mocha: {
         // change Karma's debug.html to the mocha web reporter
         reporter: "html",
-        timeout: "600000"
-      }
-    }
+        timeout: "600000",
+      },
+    },
   });
 };

sdk/communication/communication-chat/karma.conf.js

@@ -113,7 +113,7 @@
     "mocha-junit-reporter": "^2.0.0",
     "mocha": "^7.1.1",
     "nyc": "^15.0.0",
-    "prettier": "^1.16.4",
+    "prettier": "^2.5.1",
     "rimraf": "^3.0.0",
     "rollup-plugin-sourcemaps": "^0.4.2",
     "rollup-plugin-shim": "^1.0.0",

sdk/communication/communication-chat/package.json

@@ -30,13 +30,13 @@ export function nodeConfig(test = false) {
         values: {
           // replace dynamic checks with if (true) since this is for node only.
           // Allows rollup's dead code elimination to be more aggressive.
-          "if (isNode)": "if (true)"
-        }
+          "if (isNode)": "if (true)",
+        },
       }),
       nodeResolve({ preferBuiltins: true }),
       json(),
-      cjs()
-    ]
+      cjs(),
+    ],
   };
 
   if (test) {
@@ -45,7 +45,7 @@ export function nodeConfig(test = false) {
       "dist-esm/test/public/*.spec.js",
       "dist-esm/test/internal/*.spec.js",
       "dist-esm/test/public/node/*.spec.js",
-      "dist-esm/test/internal/node/*.spec.js"
+      "dist-esm/test/internal/node/*.spec.js",
     ];
     baseConfig.plugins.unshift(multiEntry({ exports: false }));
 
@@ -75,7 +75,7 @@ export function browserConfig(test = false) {
       format: "umd",
       name: "Azure.Communication.Chat",
       sourcemap: true,
-      globals: { "@azure/core-http": "Azure.Core.HTTP" }
+      globals: { "@azure/core-http": "Azure.Core.HTTP" },
     },
     preserveSymlinks: false,
     plugins: [
@@ -86,31 +86,31 @@ export function browserConfig(test = false) {
           // replace dynamic checks with if (false) since this is for
           // browser only. Rollup's dead code elimination will remove
           // any code guarded by if (isNode) { ... }
-          "if (isNode)": "if (false)"
-        }
+          "if (isNode)": "if (false)",
+        },
       }),
       shim({
         constants: `export default {}`,
         fs: `export default {}`,
         os: `export default {}`,
         dotenv: `export function config() { }`,
-        path: `export default {}`
+        path: `export default {}`,
       }),
       nodeResolve({
         mainFields: ["module", "browser"],
-        preferBuiltins: false
+        preferBuiltins: false,
       }),
       json(),
       cjs({
         namedExports: {
           chai: ["assert"],
           events: ["EventEmitter"],
           "@azure/communication-signaling": ["CommunicationSignalingClient", "SignalingClient"],
-          ...openTelemetryCommonJs()
-        }
+          ...openTelemetryCommonJs(),
+        },
       }),
-      viz({ filename: "dist-browser/browser-stats.html", sourcemap: false })
-    ]
+      viz({ filename: "dist-browser/browser-stats.html", sourcemap: false }),
+    ],
   };
 
   if (test) {
@@ -119,7 +119,7 @@ export function browserConfig(test = false) {
       "dist-esm/test/public/*.spec.js",
       "dist-esm/test/internal/*.spec.js",
       "dist-esm/test/public/browser/*.spec.js",
-      "dist-esm/test/internal/browser/*.spec.js"
+      "dist-esm/test/internal/browser/*.spec.js",
     ];
     baseConfig.plugins.unshift(multiEntry({ exports: false }));
     baseConfig.output.file = "dist-test/index.browser.js";

sdk/communication/communication-chat/rollup.base.config.js

@@ -8,7 +8,7 @@
 import { ChatClient } from "@azure/communication-chat";
 import {
   AzureCommunicationTokenCredential,
-  parseConnectionString
+  parseConnectionString,
 } from "@azure/communication-common";
 import { CommunicationIdentityClient } from "@azure/communication-identity";
 

sdk/communication/communication-chat/samples-dev/messageOperations.ts

@@ -9,7 +9,7 @@ import { ChatClient } from "@azure/communication-chat";
 import {
   AzureCommunicationTokenCredential,
   getIdentifierKind,
-  parseConnectionString
+  parseConnectionString,
 } from "@azure/communication-common";
 import { CommunicationIdentityClient } from "@azure/communication-identity";
 
@@ -42,9 +42,9 @@ export async function main() {
     participants: [
       {
         id: userSue.user,
-        displayName: "Sue"
-      }
-    ]
+        displayName: "Sue",
+      },
+    ],
   };
   await chatThreadClient.addParticipants(addParticipantsRequest);
   console.log(`Added chat participant user.`);

sdk/communication/communication-chat/samples-dev/participantsOperations.ts

@@ -8,7 +8,7 @@
 import { ChatClient } from "@azure/communication-chat";
 import {
   AzureCommunicationTokenCredential,
-  parseConnectionString
+  parseConnectionString,
 } from "@azure/communication-common";
 import { CommunicationIdentityClient } from "@azure/communication-identity";
 
@@ -35,15 +35,15 @@ export async function main() {
   // create chat thread
   console.log("Creating Thread...");
   const createChatThreadRequest = {
-    topic: "Hello, World!"
+    topic: "Hello, World!",
   };
   const createChatThreadOptions = {
     participants: [
       {
         id: user,
-        displayName: "Jack"
-      }
-    ]
+        displayName: "Jack",
+      },
+    ],
   };
   const createChatThreadResult = await chatClient.createChatThread(
     createChatThreadRequest,