@nordicsemiconductor/pc-nrfconnect-shared 230.0.0 → 231.0.0

package/nrfutil/version/version.ts

@@ -33,15 +33,15 @@ export type DiscriminatedVersion =
     | StringVersion;
 
 export const isSemanticVersion = (
-    version?: DiscriminatedVersion
+    version?: DiscriminatedVersion,
 ): version is SemanticVersion => version?.versionFormat === 'semantic';
 
 export const isIncrementalVersion = (
-    version?: DiscriminatedVersion
+    version?: DiscriminatedVersion,
 ): version is IncrementalVersion => version?.versionFormat === 'incremental';
 
 export const isStringVersion = (
-    version?: DiscriminatedVersion
+    version?: DiscriminatedVersion,
 ): version is StringVersion => version?.versionFormat === 'string';
 
 export const versionToString = (version: DiscriminatedVersion) => {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@nordicsemiconductor/pc-nrfconnect-shared",
-    "version": "230.0.0",
+    "version": "231.0.0",
     "description": "Shared commodities for developing pc-nrfconnect-* packages",
     "repository": {
         "type": "git",
@@ -53,8 +53,6 @@
         "@types/shasum": "1.0.0",
         "@types/triple-beam": "1.3.2",
         "@types/uuid": "8.3.4",
-        "@typescript-eslint/eslint-plugin": "5.57.1",
-        "@typescript-eslint/parser": "5.57.1",
         "@xterm/headless": "^5.5.0",
         "adm-zip": "^0.5.5",
         "applicationinsights": "^2.9.6",
@@ -67,14 +65,15 @@
         "esbuild": "0.19.2",
         "esbuild-sass-plugin": "2.13.0",
         "esbuild-style-plugin": "1.6.2",
-        "eslint": "8.37.0",
+        "eslint": "^8.57.1",
         "eslint-config-airbnb": "19.0.4",
         "eslint-config-prettier": "8.8.0",
         "eslint-import-resolver-typescript": "3.5.5",
         "eslint-plugin-import": "2.27.5",
+        "eslint-plugin-jsonc": "^2.20.1",
         "eslint-plugin-jsx-a11y": "6.7.1",
         "eslint-plugin-md": "^1.0.19",
-        "eslint-plugin-prettier": "^4.2.1",
+        "eslint-plugin-prettier": "^5.5.4",
         "eslint-plugin-react": "7.32.2",
         "eslint-plugin-react-hooks": "4.6.0",
         "eslint-plugin-simple-import-sort": "10.0.0",
@@ -93,7 +92,7 @@
         "nrf-intel-hex": "^1.4.0",
         "postcss": "8.4.24",
         "postcss-modules": "^6.0.0",
-        "prettier": "2.8.8",
+        "prettier": "^3.6.2",
         "prettier-plugin-tailwindcss": "^0.3.0",
         "prettysize": "2.0.0",
         "protobufjs": "^7.0.0",
@@ -117,7 +116,8 @@
         "tailwindcss": "3.3.2",
         "tree-kill-promise": "^3.0.14",
         "tsx": "^4.20.3",
-        "typescript": "4.9.5",
+        "typescript": "^5.9.2",
+        "typescript-eslint": "^8.45.0",
         "util": "0.12.5",
         "uuid": "8.3.2",
         "winston": "3.8.2",

package/release_notes.md

@@ -1,4 +1,22 @@
-### Added
+### Changed
 
--   Property `nrfConnectForDesktop.fixedSize` an app can specify, to have a
-    window with a fixed size, like the Quick Start app.
+- Updated Prettier 2.8.8 → 3.6.2
+- Updated typescript-eslint 5.57.1 → 8.45.0
+- Updated TypeScript 4.9.5 → 5.9.2
+
+### Steps to upgrade when using this package
+
+- The new version of prettier will lead to some formatting changes, especially
+  added commas and changed whitespaces, but most of those should be
+  automatically solvable by running `npm run check:lint -- --fix`.
+- Some linting rules slightly changed and you may need to update some
+  constructs. E.g. if in a `catch` block the exception is intentionally not
+  used, then you must now remove it: `try {…} catch (e) {/* Ignore */}` →
+  `try {…} catch {/* Ignore */}`.
+- Run `npm run check:types` to check if the new TypeScript version breaks the
+  type checks. As described at
+  https://www.typescriptlang.org/docs/handbook/release-notes/typescript-5-9.html#libdts-changes,
+  a typical problem occurs when you use `Buffer`. First run
+  `npm update @types/node --save-dev`, because that might already fix most
+  problems. For the rest, also first check what is written in the above linked
+  section of the release notes of TypeScript 5.9.

package/scripts/check-app-properties.ts

@@ -39,10 +39,10 @@ const mustBeEmpty = (array: string[], errorMessage: string) => {
 const mustContain = (
     existingEntries: readonly string[],
     mandatoryEntries: string[],
-    errorMessage: string
+    errorMessage: string,
 ) => {
     const missingFileEntries = mandatoryEntries.filter(
-        entry => !existingEntries.includes(entry)
+        entry => !existingEntries.includes(entry),
     );
 
     mustBeEmpty(missingFileEntries, errorMessage);
@@ -51,11 +51,11 @@ const mustContain = (
 const mustContainOneOf = (
     existingEntries: readonly string[],
     oneOfTheseEntriesIsMandatory: string[],
-    errorMessage: string
+    errorMessage: string,
 ) => {
     if (
         !oneOfTheseEntriesIsMandatory.some(entry =>
-            existingEntries.includes(entry)
+            existingEntries.includes(entry),
         )
     ) {
         fail(`${errorMessage}: ${format(oneOfTheseEntriesIsMandatory)}`);
@@ -84,7 +84,7 @@ const checkRepoUrl = (packageJson: PackageJsonApp) => {
 
     if (stripped(realGitUrl) !== stripped(declaredGitUrl)) {
         fail(
-            `package.json says the repository is located at \`${declaredGitUrl}\` but \`git remote get-url origin\` says it is at \`${realGitUrl}\`.`
+            `package.json says the repository is located at \`${declaredGitUrl}\` but \`git remote get-url origin\` says it is at \`${realGitUrl}\`.`,
         );
     }
 };
@@ -105,19 +105,19 @@ const checkFileProperty = (packageJson: PackageJsonApp) => {
     mustContain(
         packageJson.files ?? [],
         ['LICENSE', 'dist/', 'Changelog.md'],
-        'These entries are missing in the property `files` in package.json'
+        'These entries are missing in the property `files` in package.json',
     );
 
     mustContainOneOf(
         packageJson.files ?? [],
         ['resources/*', 'resources/icon.*', 'resources/'],
-        'One of these entries must be in the property `files` in package.json'
+        'One of these entries must be in the property `files` in package.json',
     );
 };
 
 const readAndCheckPackageJson = () => {
     const packageJsonResult = parsePackageJsonApp(
-        readFileSync('./package.json', 'utf8')
+        readFileSync('./package.json', 'utf8'),
     );
 
     if (!packageJsonResult.success) {
@@ -135,7 +135,7 @@ const readAndCheckPackageJson = () => {
 
 const checkChangelog = (
     packageJson: PackageJsonApp,
-    checkChangelogHasCurrentEntry: boolean
+    checkChangelogHasCurrentEntry: boolean,
 ) => {
     if (!existsSync('./Changelog.md')) {
         fail('The mandatory file `Changelog.md` is missing.');
@@ -149,7 +149,7 @@ const checkChangelog = (
         const latestChangelogEntry = getLatestEntry();
         if (!latestChangelogEntry.header.includes(packageJson.version)) {
             fail(
-                `Found no entry for the current version packageJson.version ${packageJson.version} in \`Changelog.md\`.`
+                `Found no entry for the current version packageJson.version ${packageJson.version} in \`Changelog.md\`.`,
             );
         }
     }
@@ -158,7 +158,7 @@ const checkChangelog = (
 const filesIn = (directory: string) => {
     try {
         return readdirSync(directory);
-    } catch (error) {
+    } catch {
         fail(`Unable to read directory \`${directory}\`.`);
         // Unreachable, but not understood by Typescript
         throw new Error();
@@ -169,7 +169,7 @@ const checkMandatoryResources = () => {
     mustContain(
         filesIn('./resources'),
         ['icon.svg', 'icon.icns', 'icon.ico', 'icon.png'],
-        'In the directory `resources` these files are missing'
+        'In the directory `resources` these files are missing',
     );
 };
 

package/scripts/check-for-typescript.ts

@@ -17,7 +17,7 @@ const launchNextCommand = () => {
         spawnSync(command, args, {
             shell: true,
             stdio: 'inherit',
-        }).status ?? undefined
+        }).status ?? undefined,
     );
 };
 
@@ -27,7 +27,7 @@ const assertNoTypeScriptFilesExist = () => {
     klaw('.', { filter: excludeNodeModules }).on('data', ({ path }) => {
         if (path.endsWith('.ts') || path.endsWith('.tsx')) {
             console.log(
-                "Your project contains TypeScript files (with the file ending .ts or .tsx), so it also must contain a file 'tsconfig.json'.\n"
+                "Your project contains TypeScript files (with the file ending .ts or .tsx), so it also must contain a file 'tsconfig.json'.\n",
             );
             process.exit(1);
         }

package/scripts/create-source.ts

@@ -33,12 +33,12 @@ const parseOptions = () =>
         .addOption(
             new Option('-a, --access-level <access level>', 'Access level')
                 .choices(validAccessLevels)
-                .makeOptionMandatory()
+                .makeOptionMandatory(),
         )
         .requiredOption('-n, --name <name>', 'Name, e.g. "Release Test"')
         .requiredOption(
             '-d, --description <description>',
-            'Longer description, e.g. "Versions we intend to release next"'
+            'Longer description, e.g. "Versions we intend to release next"',
         )
         .parse()
         .opts();

package/scripts/esbuild-renderer.ts

@@ -18,13 +18,13 @@ const packageJson = JSON.parse(fs.readFileSync('package.json', 'utf8'));
 
 const projectSpecificTailwindConfigPath = path.join(
     process.cwd(),
-    'tailwind.config.js'
+    'tailwind.config.js',
 );
 const tailwindConfig = () =>
     fs.existsSync(projectSpecificTailwindConfigPath)
         ? projectSpecificTailwindConfigPath
         : require.resolve(
-              '@nordicsemiconductor/pc-nrfconnect-shared/config/tailwind.config.js'
+              '@nordicsemiconductor/pc-nrfconnect-shared/config/tailwind.config.js',
           );
 
 type AdditionalOptions = Required<Pick<BuildOptions, 'entryPoints'>> &
@@ -37,7 +37,7 @@ const outfileOrDir = (additionalOptions: AdditionalOptions) =>
 
 const options = (
     additionalOptions: AdditionalOptions,
-    externalReact: boolean
+    externalReact: boolean,
 ) =>
     ({
         format: 'iife',
@@ -98,7 +98,7 @@ const options = (
                         const [, shortpath] = filter.exec(args.path)!; // eslint-disable-line @typescript-eslint/no-non-null-assertion
                         const resolvedPath = `${path.join(
                             args.resolveDir,
-                            shortpath
+                            shortpath,
                         )}r`;
                         return { path: resolvedPath };
                     });
@@ -107,7 +107,7 @@ const options = (
                         const filePath = args.path.replace('.svgr', '.svg');
                         const svg = await fs.promises.readFile(
                             filePath,
-                            'utf8'
+                            'utf8',
                         );
                         const plugins = ['@svgr/plugin-jsx'];
                         const contents = await svgr(svg, { plugins });
@@ -120,15 +120,15 @@ const options = (
             },
         ],
         ...additionalOptions,
-    } satisfies BuildOptions);
+    }) satisfies BuildOptions;
 
 export const build = async (
     additionalOptions: AdditionalOptions,
-    { externalReact = false } = {}
+    { externalReact = false } = {},
 ) => {
     if (process.argv.includes('--watch')) {
         const context = await esbuild.context(
-            options(additionalOptions, externalReact)
+            options(additionalOptions, externalReact),
         );
 
         await context.rebuild();

package/scripts/esbuild.ts

@@ -69,9 +69,9 @@ const copyFiles = () => {
 
         fs.copyFileSync(
             fileInShared(
-                'fw/bootloader/graviton_bootloader_v1.0.1-[nRF5_SDK_15.0.1-1.alpha_f76d012].zip'
+                'fw/bootloader/graviton_bootloader_v1.0.1-[nRF5_SDK_15.0.1-1.alpha_f76d012].zip',
             ),
-            './fw/graviton_bootloader_v1.0.1-[nRF5_SDK_15.0.1-1.alpha_f76d012].zip'
+            './fw/graviton_bootloader_v1.0.1-[nRF5_SDK_15.0.1-1.alpha_f76d012].zip',
         );
     }
 };

package/scripts/get-release-numbers.ts

@@ -18,7 +18,7 @@ const getLatestReleaseName = () => {
 export default (latestReleaseName = getLatestReleaseName()) => {
     const latest = Number(
         /^v(?<versionNumber>\d+)$/.exec(latestReleaseName)?.groups
-            ?.versionNumber
+            ?.versionNumber,
     );
 
     return { latest, next: latest + 1 };

package/scripts/is-releasable.ts

@@ -25,7 +25,7 @@ const fail = (message: string) => {
 const assertPackageJsonIsCorrect = (expectedVersionNumber: number) => {
     if (packageJson.version !== `${expectedVersionNumber}.0.0`) {
         fail(
-            `Version number in package.json must be '${expectedVersionNumber}.0.0' but is '${packageJson.version}'`
+            `Version number in package.json must be '${expectedVersionNumber}.0.0' but is '${packageJson.version}'`,
         );
     }
 };
@@ -44,7 +44,7 @@ const assertLatestHeaderIsCorrect = (expectedVersionNumber: number) => {
 
     if (expectedHeaderline !== actualHeaderline) {
         fail(
-            `Latest entry in Changelog.md is not as expected:\n  Expected: ${expectedHeaderline}\n  Actual:   ${actualHeaderline}`
+            `Latest entry in Changelog.md is not as expected:\n  Expected: ${expectedHeaderline}\n  Actual:   ${actualHeaderline}`,
         );
     }
 };
@@ -61,7 +61,7 @@ const main = () => {
     assertChangelogIsCorrect(releaseNumbers.next);
 
     console.log(
-        `The currently released version is ${releaseNumbers.latest}, so the next one will be ${releaseNumbers.next}.\n`
+        `The currently released version is ${releaseNumbers.latest}, so the next one will be ${releaseNumbers.next}.\n`,
     );
 };
 

package/scripts/nordic-publish.ts

@@ -37,17 +37,17 @@ let client: Client;
 abstract class Client {
     abstract sourceUrl: string;
 
-    initialise(options: Options): Promise<void> | void {} // eslint-disable-line @typescript-eslint/no-unused-vars
+    initialise(options: Options): Promise<void> | void {} // eslint-disable-line no-empty-function,@typescript-eslint/no-unused-vars
     abstract end(): void;
 
     abstract download(filename: string): Promise<string>;
     abstract uploadContent(
-        content: Buffer,
-        remoteFilename: string
+        content: Buffer<ArrayBuffer>,
+        remoteFilename: string,
     ): Promise<void>;
     abstract uploadLocalFile(
         localFilename: string,
-        remoteFilename: string
+        remoteFilename: string,
     ): Promise<void>;
 }
 
@@ -89,7 +89,7 @@ class FtpClient extends Client {
     connect = () =>
         new Promise<void>((resolve, reject) => {
             console.log(
-                `Connecting to ftp://${this.user}@${this.host}:${this.port}`
+                `Connecting to ftp://${this.user}@${this.host}:${this.port}`,
             );
             this.ftpClient.once('error', err => {
                 this.ftpClient.removeAllListeners('ready');
@@ -115,8 +115,8 @@ class FtpClient extends Client {
                     reject(
                         new Error(
                             '\nError: Failed to change to directory. ' +
-                                'Check whether it exists on the FTP server.'
-                        )
+                                'Check whether it exists on the FTP server.',
+                        ),
                     );
                 } else {
                     resolve();
@@ -145,12 +145,12 @@ class FtpClient extends Client {
 
     upload = (
         contentOrLocalFilename: string | Buffer,
-        remoteFilename: string
+        remoteFilename: string,
     ) =>
         new Promise<void>((resolve, reject) => {
             console.log(`Uploading file ${remoteFilename}`);
             this.ftpClient.put(contentOrLocalFilename, remoteFilename, err =>
-                err ? reject(err) : resolve()
+                err ? reject(err) : resolve(),
             );
         });
 
@@ -176,7 +176,7 @@ class ArtifactoryClient extends Client {
 
         if (this.token == null) {
             throw new Error(
-                'The environment variable ARTIFACTORY_TOKEN must be set.'
+                'The environment variable ARTIFACTORY_TOKEN must be set.',
             );
         }
 
@@ -220,11 +220,11 @@ class ArtifactoryClient extends Client {
         if (res.ok || res.status === 404) return;
 
         this.filesWhereCacheZappingFailed.push(
-            `${this.folderName}/${filename}`
+            `${this.folderName}/${filename}`,
         );
     };
 
-    upload = async (content: Buffer, remoteFilename: string) => {
+    upload = async (content: Buffer<ArrayBuffer>, remoteFilename: string) => {
         const url = `${this.uploadUrl}/${remoteFilename}`;
         const res = await fetch(url, {
             method: 'PUT',
@@ -239,7 +239,7 @@ class ArtifactoryClient extends Client {
         await this.zapCache(remoteFilename);
     };
 
-    uploadContent = (content: Buffer, remoteFilename: string) => {
+    uploadContent = (content: Buffer<ArrayBuffer>, remoteFilename: string) => {
         console.log(`Uploading content for ${remoteFilename}`);
 
         return this.upload(content, remoteFilename);
@@ -247,7 +247,7 @@ class ArtifactoryClient extends Client {
 
     uploadLocalFile = (localFilename: string, remoteFilename: string) => {
         console.log(
-            `Uploading local file ${localFilename} as ${remoteFilename}`
+            `Uploading local file ${localFilename} as ${remoteFilename}`,
         );
 
         return this.upload(fs.readFileSync(localFilename), remoteFilename);
@@ -256,14 +256,14 @@ class ArtifactoryClient extends Client {
     end = () => {
         if (this.filesWhereCacheZappingFailed.length > 0) {
             console.warn(
-                `\nCache zapping failed for these files, probably because your Artifactory token lacks permission for it:`
+                `\nCache zapping failed for these files, probably because your Artifactory token lacks permission for it:`,
             );
             this.filesWhereCacheZappingFailed.forEach(file => {
                 console.warn(`- ${file}`);
             });
 
             console.warn(
-                '\nGo to https://github.com/NordicSemiconductor/pc-nrfconnect-shared/actions/workflows/zap-cache.yml, run the workflow and paste this string as the list of paths to zap them manually:'
+                '\nGo to https://github.com/NordicSemiconductor/pc-nrfconnect-shared/actions/workflows/zap-cache.yml, run the workflow and paste this string as the list of paths to zap them manually:',
             );
             console.warn('  ', this.filesWhereCacheZappingFailed.join(', '));
         }
@@ -290,7 +290,7 @@ const isAccessLevel = (value: string): value is AccessLevel =>
 
 const splitSourceAndAccessLevel = (sourceAndMaybeAccessLevel: string) => {
     const match = sourceAndMaybeAccessLevel.match(
-        /(?<source>.*?)\s*\((?<accessLevel>.*)\)/
+        /(?<source>.*?)\s*\((?<accessLevel>.*)\)/,
     );
 
     if (match == null) {
@@ -302,8 +302,8 @@ const splitSourceAndAccessLevel = (sourceAndMaybeAccessLevel: string) => {
     if (!isAccessLevel(accessLevel)) {
         throw new Error(
             `The specified access level "${accessLevel}" must be one of ${validAccessLevels.join(
-                ', '
-            )}.`
+                ', ',
+            )}.`,
         );
     }
 
@@ -326,19 +326,19 @@ const parseOptions = (): Options => {
             '-s, --source <source>',
             'Specify the source to publish (e.g. "official" or "releast-test"). ' +
                 'When publishing to Artifactory, an access level can be ' +
-                'specified at the end in parantheses (e.g. "official (external)").'
+                'specified at the end in parantheses (e.g. "official (external)").',
         )
         .addOption(
             new Option(
                 '-d, --destination <ftp|artifactory>',
-                'Specify where to publish.'
+                'Specify where to publish.',
             )
                 .choices(['ftp', 'artifactory'])
-                .makeOptionMandatory()
+                .makeOptionMandatory(),
         )
         .option(
             '-n, --no-pack',
-            'Publish existing .tgz file at the root directory without npm pack.'
+            'Publish existing .tgz file at the root directory without npm pack.',
         )
         .parse();
 
@@ -377,8 +377,8 @@ const getShasum = (filePath: string) => {
     } catch (error) {
         throw new Error(
             `Unable to read file when verifying shasum: ${filePath}. \nError: ${errorAsString(
-                error
-            )}`
+                error,
+            )}`,
         );
     }
 };
@@ -413,14 +413,14 @@ const packOrReadPackage = (options: Options): App => {
 
 const assertAppVersionIsValid = (
     latestAppVersion: string | undefined,
-    app: App
+    app: App,
 ) => {
     if (latestAppVersion != null) {
         console.log(`Latest published version ${latestAppVersion}`);
 
         if (semver.lte(app.version, latestAppVersion) && app.isOfficial) {
             throw new Error(
-                'Current package version cannot be published, bump it higher'
+                'Current package version cannot be published, bump it higher',
             );
         }
     }
@@ -438,7 +438,7 @@ const downloadSourceJson = async () => {
             (sourceJson.apps !== undefined && !Array.isArray(sourceJson.apps))
         ) {
             throw new Error(
-                '`source.json` does not have the expected content.'
+                '`source.json` does not have the expected content.',
             );
         }
 
@@ -461,21 +461,21 @@ const getUpdatedSourceJson = async (app: App): Promise<SourceJson> => {
         ...sourceJson,
         apps: [
             ...new Set(sourceJson.apps).add(
-                `${app.sourceUrl}/${app.appInfoName}`
+                `${app.sourceUrl}/${app.appInfoName}`,
             ),
         ].sort(),
     };
 };
 
 const downloadExistingAppInfo = async (
-    app: App
+    app: App,
 ): Promise<Partial<Pick<AppInfo, 'latestVersion' | 'versions'>>> => {
     try {
         const appInfoContent = await client.download(app.appInfoName);
         return JSON.parse(appInfoContent) as AppInfo;
     } catch (error) {
         console.log(
-            `No previous app versions found due to: ${errorAsString(error)}`
+            `No previous app versions found due to: ${errorAsString(error)}`,
         );
 
         return {};
@@ -484,7 +484,7 @@ const downloadExistingAppInfo = async (
 
 const failBecauseOfMissingProperty = () => {
     throw new Error(
-        'This must never happen, because the properties were already checked before'
+        'This must never happen, because the properties were already checked before',
     );
 };
 
@@ -529,13 +529,13 @@ const getUpdatedAppInfo = async (app: App): Promise<AppInfo> => {
 const uploadSourceJson = (sourceJson: SourceJson) =>
     client.uploadContent(
         Buffer.from(JSON.stringify(sourceJson, undefined, 2)),
-        'source.json'
+        'source.json',
     );
 
 const uploadAppInfo = (app: App, appInfo: AppInfo) =>
     client.uploadContent(
         Buffer.from(JSON.stringify(appInfo, undefined, 2)),
-        app.appInfoName
+        app.appInfoName,
     );
 
 const uploadPackage = (app: App) =>

package/scripts/nrfconnect-license.ts

@@ -106,7 +106,7 @@ const asRegexp = (string: string) =>
             .replace(SPECIAL_REGEXP_CHARACTERS, '\\$1')
             .replace(LINE_ENDINGS, '\\r?\\n') // Because reference string and input string could have different EOLs
             .replace('__YEAR__', '\\d{4}')
-            .replace('__YEARS__', '(?<oldStartingYear>\\d{4})( - \\d{4})?')}`
+            .replace('__YEARS__', '(?<oldStartingYear>\\d{4})( - \\d{4})?')}`,
     );
 
 const checkLicense = () => {
@@ -153,7 +153,7 @@ const allSourceFiles = () =>
                 '.git',
             ],
             baseNameMatch: true,
-        }
+        },
     );
 
 const missesCorrectHeader = (file: string) =>
@@ -178,7 +178,7 @@ const checkHeaders = () => {
 
         fail(
             `These files do not contain the correct license, try fixing this` +
-                ` with running 'npx ${nrfconnectLicenseScript}':\n${listOfFiles}`
+                ` with running 'npx ${nrfconnectLicenseScript}':\n${listOfFiles}`,
         );
     }
 };
@@ -242,7 +242,7 @@ const updateHeader = (file: string) => {
     const COMMENT_START = '/*';
     if (!hadOutdatedLicense && remainingContent.startsWith(COMMENT_START)) {
         console.warn(
-            `The file '${file}' has a comment at the beginning which is not a known outdated license. Not touching this file.`
+            `The file '${file}' has a comment at the beginning which is not a known outdated license. Not touching this file.`,
         );
         return;
     }
@@ -254,12 +254,12 @@ const updateHeader = (file: string) => {
 
     writeFileSync(
         file,
-        shebangLine + newHeader + lineEnding + lineEnding + remainingContent
+        shebangLine + newHeader + lineEnding + lineEnding + remainingContent,
     );
     console.log(
         `${
             hadOutdatedLicense ? 'Updated' : 'Added'
-        } license header in file '${file}'`
+        } license header in file '${file}'`,
     );
 };