Skip to content

[BUG] Includes starting with trailing ./ causes to fail #544

@cs-clarence

Description

@cs-clarence

Describe the bug
When using Cocogitto in a monorepo setup, package configuration entries in include = [] that start with a leading ./, changes are not detected correctly. As a result, cog fails to recognize changes in those paths, when using the bump command:

cog bump --dry-run --auto

Result:

No conventional commits found for your packages that required a bump. Changelogs will be updated on the next bump.
Pre-Hooks and Post-Hooks have been skipped.

Removing the ./ in the entries to include:

[packages."@autorelease-test/package-a"]
path = "./package-a"
include = ["./package-a/**"]
public_api = true

To

[packages."@autorelease-test/package-a"]
path = "./package-a"
include = ["package-a/**"] # remove trailing ./
public_api = true

Now bump behaves properly:

$ cog bump --dry-run --auto
@autorelease-test/package-a-v0.1.0
v0.1.0

To Reproduce

  1. Clone the repo: https://github.com/cs-clarence/autorelease-test
  2. Open the dir with terminal
  3. Run cog bump --dry-run --auto

Expected behavior
Should still recognize changes even with leading ./ in include paths

Additional context
OS: Fedora Linux 43 (KDE Plasma Desktop Edition) x86_64

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions