From 3c69fed0836f2a0d65514172125cb1054c2e75e1 Mon Sep 17 00:00:00 2001
From: Bazyli Brzoska <bazyli.brzoska@gmail.com>
Date: Sat, 4 May 2024 20:08:24 -0700
Subject: [PATCH] fix: respect `tagSeparator` and `includeVInTag` everywhere

fixes #2282
---
 src/manifest.ts             | 7 ++++++-
 src/strategies/php-yoshi.ts | 7 ++++++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/manifest.ts b/src/manifest.ts
index be5c0ed96..8f80cde30 100644
--- 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
index fddf2c203..150617548 100644
--- 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 {