Cope with missing version attribute on module

bazel-contrib · Jan 26, 2024 · a430919 · a430919
1 parent e765a50
commit a430919
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/e2e/fixtures/unversioned/MODULE.bazel b/e2e/fixtures/unversioned/MODULE.bazel
@@ -1,4 +1 @@
-module(
-    name = "unversioned",
-    version = "0.0.0",
-)
+module(name = "unversioned")
diff --git a/src/domain/create-entry.ts b/src/domain/create-entry.ts
@@ -231,7 +231,8 @@ export class CreateEntryService {
   ): void {
     if (moduleFile.version !== version) {
       console.log(
-        "Archived MODULE.bazel version does not match release version. Creating a version patch."
+        `Archived MODULE.bazel version ${moduleFile.version} does not match release version ${version}.`,
+        "Creating a version patch.",
       );
       const patchFileName = "module_dot_bazel_version.patch";
       const existingContent = moduleFile.content;

diff --git a/src/domain/module-file.ts b/src/domain/module-file.ts
@@ -23,8 +23,8 @@ export class ModuleFile {
 
   public get version(): string {
     const regex = /module\([^)]*?version\s*=\s*"(.+?)"/s;
-    const version = this.moduleContent.match(regex)[1];
-    return version;
+    const match = this.moduleContent.match(regex);
+    return match ? match[1] : "(unversioned)";
   }
 
   public get content(): string {