Skip to content

Generic updater: x-release-please-major does not match strings prefixed with 'v' (e.g., 'v<major>') #2421

@faradayfan

Description

@faradayfan

Generic updater: x-release-please-major does not match strings prefixed with 'v' (e.g., 'v')

Environment details

  • Encountered in the release-please-action@v4
  • release-please version: 16.12.0

Steps to reproduce

When annotating a file with x-release-please-major like this:

Major: 2 # x-release-please-major
Major with v: v2 # x-release-please-major

The result is the following update:

Major: 3 # x-release-please-major
Major with v: v2 # x-release-please-major

The expected output is:

Major: 3 # x-release-please-major
Major with v: v3 # x-release-please-major

Looking at the generic updater code, this appears to be directly caused by the regex used for single version updates. It looking for a preceding word boundary, which in this case there is none.

I was unable to find documentation on why this is a desired behavior, or is this was fixed in a more recent version.

Metadata

Metadata

Assignees

Labels

priority: p3Desirable enhancement or fix. May not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions