Page MenuHomePhorge

Co-Author support in arc land T15044 (Arcanist repo)
Needs RevisionPublic

Authored by TitanNano on Dec 2 2021, 16:24.
Referenced Files
F2917387: D25029.1737506437.diff
Tue, Jan 21, 00:40
F2917153: D25029.1737501973.diff
Mon, Jan 20, 23:26
F2916784: D25029.1737501722.diff
Mon, Jan 20, 23:22
F2914900: D25029.1737444751.diff
Mon, Jan 20, 07:32
F2914729: D25029.1737444487.diff
Mon, Jan 20, 07:28
F2902161: D25029.1737297052.diff
Sat, Jan 18, 14:30
F2901525: D25029.1737281563.diff
Sat, Jan 18, 10:12
F2893329: D25029.1737224072.diff
Fri, Jan 17, 18:14

Details

Reviewers
avivey
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15044: Co-Author support in arc land
Summary

We have to parse the commit messages and add all co-authors to the "local:commits"
diff property, so we can use it later during arc land.

See also D25028.

Test Plan
  1. create a commit with one or more "Co-authored-by" lines.
  2. arc diff the commit into a revision
  3. verify that the local:commits property contains a coAuthors array with all co-authors

Diff Detail

Repository
rARC Arcanist
Branch
issues/T15044
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 58
Build 58: arc lint + arc unit

Event Timeline

trim whitespace from name and email

avivey edited the summary of this revision. (Show Details)
avivey subscribed.
avivey added inline comments.
src/repository/api/ArcanistGitAPI.php
224

Looks like we already upload the entire commit message - can we do this logic in the server, avoiding adding this key to the API?

avivey requested changes to this revision.Mar 30 2024, 12:28

house-keeping...

This revision now requires changes to proceed.Mar 30 2024, 12:28
Cigaryno retitled this revision from Co-Author support in arc land T15044 to Co-Author support in arc land T15044 (Arcanist repo).Wed, Jan 1, 15:14