Skip to content

chore: default draft tag message to version name MONGOSH-3554 - #2816

Merged
dudaschar merged 1 commit into
mainfrom
MONGOSH-3554
Aug 5, 2026
Merged

chore: default draft tag message to version name MONGOSH-3554#2816
dudaschar merged 1 commit into
mainfrom
MONGOSH-3554

Conversation

@dudaschar

Copy link
Copy Markdown
Collaborator

If the tag creation is signed in, the process was hanging locally because git was waiting for a message confirmation.
This change defaults the message to be the tag name, if necessary.

@dudaschar dudaschar self-assigned this Aug 4, 2026
Copilot AI lite review requested due to automatic review settings August 4, 2026 16:18
@dudaschar
dudaschar requested a review from a team as a code owner August 4, 2026 16:18
@dudaschar
dudaschar requested a review from nbbeeken August 4, 2026 16:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the local release-draft helper to create git tags non-interactively by defaulting the tag message to the version/tag name, addressing hangs when git tag signing triggers an editor prompt.

Changes:

  • Create draft tags using git tag -m <tag> <tag> so a message is always provided.
  • Update unit tests to assert the new git tag argument list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/build/src/local/trigger-release-draft.ts Adds -m <tag> when creating the draft tag to avoid interactive editor prompts.
packages/build/src/local/trigger-release-draft.spec.ts Updates assertions to match the new git tag invocation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 87 to 91
console.info('... creating and pushing tag ...');
spawnSync('git', ['tag', nextTagName], {
spawnSync('git', ['tag', '-m', nextTagName, nextTagName], {
cwd: repositoryRoot,
encoding: 'utf-8',
});

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll wait to actually publish to see if it's a real concern before updating.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, this will take a while so I will merge and if we need we can open another PR.

@dudaschar
dudaschar merged commit b3f1892 into main Aug 5, 2026
136 of 145 checks passed
@dudaschar
dudaschar deleted the MONGOSH-3554 branch August 5, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants