Fails to detect letter case with gitmoji prefixed commit messages

[ttytm]

Is there an existing issue for this?

• I have searched the existing issues

Package ecosystem

cargo

Package manager version

No response

Language version

No response

Manifest location and content before the Dependabot update

No response

dependabot.yml content

version: 2
updates:
  - package-ecosystem: cargo
    directory: /
    schedule:
      interval: weekly
      time: "04:00"
    assignees:
      - tobealive
    commit-message:
      prefix: "⬆️ "

  - package-ecosystem: github-actions
    directory: /
    schedule:
      interval: weekly
      time: "04:00"
    assignees:
      - tobealive
    commit-message:
      prefix: "⬆️ "

Updated dependency

No response

What you expected to see, versus what you actually saw

Dependabot won't detect the letter case after merging previous commits with lowercase.

Commit message submitted by dependabot:

⬆️ Bump h2 from 0.3.16 to 0.3.17

Merged message:

⬆️ bump h2 from 0.3.16 to 0.3.17

Consecutive commits:

⬆️ Bump h2 from 0.3.16 to 0.3.17

Native package manager behavior

No response

Images of the diff or a link to the PR, issue, or logs

This project uses the gitmoji code for its commit prefixes where the problem occurs.

https://github.com/tobealive/wthrr-the-weathercrab

Related issues I've searched and read include it's linked relatives:

• commit messages break conventional commit rules feedback#292
• build(devDeps): bump ... violates config-conventional feedback#74

Smallest manifest that reproduces the issue

No response

[jeffwidman]

Thanks, I didn't fully understand what you were asking for until I read:

• commit messages break conventional commit rules feedback#292.

This is a fairly rare edgecase, so it's tough to justify the core team working on it. Thankfully, this is open source, so if you or anyone else is interested, we'd welcome a PR.

I suspect this one shouldn't be that hard to fix, especially if you use one of our debugging tools to simulate the update job locally: https://github.com/dependabot/dependabot-core#debugging-problems

You can add a few debugger statements to narrow down where it's not picking up the correct casing: https://github.com/dependabot/dependabot-core#adding-debug-breakpoints

[kevinmatthes]

I had a look at the source code and I think the cause of this problem is https://github.com/dependabot/dependabot-core/blob/main/common/lib/dependabot/pull_request_creator/pr_name_prefixer.rb#L132. The easiest solution should be to just invert the boolean there in case the GitMoji style is used; but this is just an assumption which I never tested.

However, there might be some projects which use GitMoji and require an upper case B for "Bump". Thus, I would like to suggest to add a general configuration option controlling the case of "Bump" such that the behaviour of the case detection could be explicitly controlled regardless of the exact commit message style. I assume such a feature would be useful for a broader audience and would not break anything when setting the current behaviour as default in case nothing is configured. Furthermore, with the case of "Bump" set explicitly, the performance would be enhanced as the case detection could be completed faster.