Skip to content
This repository has been archived by the owner on Dec 20, 2022. It is now read-only.

Set author and remove review when a branch is merged #19

Closed
adiroiban opened this issue Jan 27, 2015 · 2 comments
Closed

Set author and remove review when a branch is merged #19

adiroiban opened this issue Jan 27, 2015 · 2 comments

Comments

@adiroiban
Copy link
Member

As commit messages should be in the format

Merge <branch name>: Brief description

Author: <comma_separated_names>
Reviewer: <comma_separated_names>
Fixes: #<ticket number>

we should be able to extract the author from commit message.

I assume that we can change the doc to state that main author should be listed first.

I assume that I will need to work on this

https://github.com/twisted-infra/trac-config/blob/master/trac-post-commit-hook


Is there a staging trac/svn instance where we can test this?

For my project I have a staging trac instance so I can use it during development and testing and will write unit tests

@adiroiban
Copy link
Member Author

I will wait for #20 be fixed before continuing to work on this as we might use a different code for the commit hook

@adiroiban
Copy link
Member Author

twisted-infra/braid#124

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant