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

Respect GIT_AUTHOR_*/GIT_COMMITTER_* environment variables #925

Merged
merged 1 commit into from
Jun 14, 2017

Conversation

BinaryMuse
Copy link
Contributor

GIT_AUTHOR_* and GIT_COMMITTER_* environment variables weren't being sent to the Git worker process, so pairing information (such as set by git-pear) didn't work.

/cc @smashwilson
/cc @jasonrudolph for info

@BinaryMuse BinaryMuse requested a review from smashwilson June 14, 2017 21:04
Copy link
Contributor

@smashwilson smashwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aye aye, this should do the trick!

env[envVar] = process.env[envVar];
}
});

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like I've done this a few times now. It might be worth refactoring into a helper somewhere 🤔

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I think the other times are in #846 which isn't :shipit: yet. And which would be run from a process that can't import helpers easily.

@BinaryMuse BinaryMuse merged commit d7400d0 into master Jun 14, 2017
@BinaryMuse BinaryMuse deleted the mkt-respect-author-committer-env branch June 14, 2017 21:55
BinaryMuse pushed a commit that referenced this pull request Aug 1, 2017
Respect GIT_AUTHOR_*/GIT_COMMITTER_* environment variables
(cherry picked from commit d7400d0)
@BinaryMuse BinaryMuse mentioned this pull request Aug 1, 2017
25 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants