I uploaded this revision with arc diff
- Group Reviewers
- rHGb2c27d84f05c: phabricator: include the suggested arc config in the repo
This looks good to me too.
Personally I would vote for setting history.immutable to false too. This mostly affects the behavior of the arc diff and arc amend commands. This probably won't matter at all for people that submit diffs using the phabsend extension instead of arc diff.
When set to false, arc diff will rewrite your local commit message to include the differential revision ID when submitting a diff. arc amend will update your commit message to add a Reviewed By line once your diff has been accepted.
Personally I find it useful to have this information in my local commit messages. (The phabdiff and phabstatus extensions in the rFBHGEXT repository can also report some useful information if they find this field present in your commit message.)
The only downside of leaving history.immutable as true is that you don't get this information in your commit message when running arc diff / arc amend; it shouldn't impact any other behavior.
One arguable downside of setting history.immutable to false is that your local commit IDs do change after running arc diff or arc amend if it needed to update your commit message.