Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release Please cannot parse any commits have merge commit with Typo not following Conventional Commits Format #1064

Open
aaronware opened this issue Dec 11, 2024 · 0 comments
Labels
priority: p3 Desirable enhancement or fix. May not be included in next release. type: question Request for information or clarification. Not an issue.

Comments

@aaronware
Copy link

Thanks for stopping by to ask us a question! Please make sure to include:

  • What you're trying to do

Create a release

  • What code you've already tried

Standard Release please using conventional commits format

  • Any error messages you're getting

release-please failed: Cannot read properties of undefined (reading 'startsWith')

I've tried committing again with a proper conventional commit message.

Additionally, I tried to lower the release search depth and commit search depth in the config and while I see it in the output/echo it's still looking at 25+ commits before it finally throw that it can't parse ANY commits, however this started when I accidentally had a typo in my commit message

❯ release search depth: 1
❯ Fetching releases with cursor undefined
❯ Found release for path ., v1.0.7
❯ release for path: ., version: 1.0.7, sha: d71b326762dda80ef918f33c354096bd955bfd60
✔ Collecting commits since all latest releases
❯ commit search depth: 1
❯ Set(1) { 'd71b326762dda80ef918f33c354096bd955bfd60' }
❯ Fetching merge commits on branch main with cursor: undefined
...

...
❯ commit could not be parsed: ef89574b0415ad615c5662d7683c8446007c84ed Merge pull request #99 from linchpin/issue/LINCHPIN-4178
⚠ No latest release pull request found.
❯ commit could not be parsed: ca21789c498d5394d194306306ca35786570497e Merge pull request #96 from linchpin/renovate/npm-frontend
❯ commit could not be parsed: 72a7d2357dd52ad1fdbdc4147ad1f5b6cae8b5c7 Merge pull request #97 from linchpin/renovate/major-wordpress-monorepo
❯ commit could not be parsed: 1bcd20be531c7249fb71588cf0b564276788d615 Merge pull request #95 from linchpin/renovate/npm-dev-dependencies
❯ commits: 15
✔ Considering: 15 commits
❯ component:
❯ pull request title pattern: undefined

Any help would be greatly appreciated

@aaronware aaronware added priority: p3 Desirable enhancement or fix. May not be included in next release. type: question Request for information or clarification. Not an issue. labels Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p3 Desirable enhancement or fix. May not be included in next release. type: question Request for information or clarification. Not an issue.
Projects
None yet
Development

No branches or pull requests

1 participant