fix: respect tagSeparator and includeVInTag everywhere

fixes googleapis#2282
niieani · May 5, 2024 · 3c69fed · 3c69fed
1 parent cb106cd
commit 3c69fed
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/manifest.ts b/src/manifest.ts
@@ -688,7 +688,12 @@ export class Manifest {
           `No latest release found for path: ${path}, component: ${component}, but a previous version (${version.toString()}) was specified in the manifest.`
         );
         releasesByPath[path] = {
-          tag: new TagName(version, component),
+          tag: new TagName(
+            version,
+            component,
+            this.repositoryConfig[path].tagSeparator,
+            this.repositoryConfig[path].includeVInTag
+          ),
           sha: '',
           notes: '',
         };

diff --git a/src/strategies/php-yoshi.ts b/src/strategies/php-yoshi.ts
@@ -92,7 +92,12 @@ export class PHPYoshi extends BaseStrategy {
     const versionsMap: VersionsMap = new Map();
     const directoryVersionContents: Record<string, ComponentInfo> = {};
     const component = await this.getComponent();
-    const newVersionTag = new TagName(newVersion, component);
+    const newVersionTag = new TagName(
+      newVersion,
+      component,
+      this.tagSeparator,
+      this.includeVInTag
+    );
     let releaseNotesBody = `## ${newVersion.toString()}`;
     for (const directory of topLevelDirectories) {
       try {