nextcloud-libraries / nextcloud-libraries/rector

Conventional commit check is preventing "Assisted by:" footer

Open
#104 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
PHP
Stars
3
Forks
1
Avg merge
11m
Merged PRs (30d)
1

Description

commit-msg: 

                                                                                                                        
 [ERROR] Invalid Commit Message: Footer values may not contain other footers.                                           
                                                                                                                        
         The commit message is not properly formatted according to the Conventional Commits specification. For more     
         details, see https://www.conventionalcommits.org/en/v1.0.0/                                                    
                                                                                                                        

 - \Ramsey\CaptainHook\ValidateConventionalCommit                    : failed
captainhook failed executing all actions, took: 0.03s
1 action failed

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the commit-msg hook or CaptainHook configuration that invokes Ramsey\CaptainHook\ValidateConventionalCommit, then check how an “Assisted by:” footer is parsed against the Conventional Commits specification. Done means commits containing that footer pass validation while incorrectly nested or malformed footers continue to fail.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.