New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Aaron to the mailmap. Woo! #9382

Merged
merged 1 commit into from Jan 7, 2018

Conversation

Projects
None yet
4 participants
@pbarry-r7
Contributor

pbarry-r7 commented Jan 7, 2018

Include @asoto-r7 in the mailmap. 馃樃

@bcoles

bcoles approved these changes Jan 7, 2018

Looks legit

@wchen-r7 wchen-r7 merged commit 340cd66 into rapid7:master Jan 7, 2018

2 checks passed

Metasploit Automation - Test Execution Successfully ran `autoPayloadTest.py`.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

wchen-r7 added a commit that referenced this pull request Jan 7, 2018

@wchen-r7

This comment has been minimized.

Show comment
Hide comment
@wchen-r7

wchen-r7 Jan 7, 2018

Contributor

Release Notes

Adding new member Aaron Soto the mailmap.

Contributor

wchen-r7 commented Jan 7, 2018

Release Notes

Adding new member Aaron Soto the mailmap.

@pbarry-r7

This comment has been minimized.

Show comment
Hide comment
@pbarry-r7

pbarry-r7 Jan 8, 2018

Contributor

I'ma readd the "no release notes" label, as it seems that this isn't something customers would be interested in (and I don't believe we've published notes on changes to the .mailmap file in the past). If I'm in error here, by all means let me know. :)

Contributor

pbarry-r7 commented Jan 8, 2018

I'ma readd the "no release notes" label, as it seems that this isn't something customers would be interested in (and I don't believe we've published notes on changes to the .mailmap file in the past). If I'm in error here, by all means let me know. :)

@mkienow-r7

This comment has been minimized.

Show comment
Hide comment
@mkienow-r7

mkienow-r7 Jan 8, 2018

Contributor

@pbarry-r7 It was a reaction to @wchen-r7 adding release notes, however, you make a great point that I didn't think of when I removed the label. :)

Contributor

mkienow-r7 commented Jan 8, 2018

@pbarry-r7 It was a reaction to @wchen-r7 adding release notes, however, you make a great point that I didn't think of when I removed the label. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment