missing space in PR title does not get caught

[scarf005]

Describe the bug

from cataclysmbnteam/Cataclysm-BN#4329

semantic.yml contents

enabled: true
titleOnly: true
targetUrl: 'https://docs.cataclysmbn.org/en/contribute/changelog_guidelines/'
types:
  - feat
  - fix
  - docs
  - style
  - refactor
  - perf
  - test
  - build
  - ci
  - chore
  - revert

# irrelevant configs skipped   

https://github.com/cataclysmbnteam/Cataclysm-BN/blob/59eb28f4a7404d1ec2f4be1eaa387d4317a8a812/.github/semantic.yml

Sample PR Title(s) (if relevant)

feat(content):Cooking with insect eggs

Sample Commit Message(s) (if relevant)

No response

[dhrapson]

We are hitting this also. Is this app still being supported in terms of code changes?

[Ezard]

Hey @scarf005 @dhrapson,

Apologies for the slow response - life has been busy recently!

This app uses @conventional-commits/parser to check whether a PR title is semantic or not (see https://github.com/Ezard/semantic-prs/blob/master/functions%2Fsrc%2Fis-message-semantic.ts#L30) - might be worth seeing if the maintainers of that package are open to adding this extra check?

If not then I can write my own parsing function for this app 🙂

[scarf005]

looks like it's tracked in @conventional-commits/parser conventional-commits/parser#47, but the repo is unmaintained since 2021 - I'm willing to open a PR with custom validator function.

[Ezard]

@dhrapson @scarf005 finally got some time over the holiday period to sort this; it's now been deployed 🙂

(thanks to @scarf005 for providing the fix!)