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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update drGroves' OpenPGP Keys #110

Merged
merged 2 commits into from Nov 2, 2017

Conversation

Projects
None yet
3 participants
@drGrove
Member

drGrove commented Nov 1, 2017

No description provided.

@drGrove drGrove referenced this pull request Nov 1, 2017

Closed

Unuseable keys listed in vault_passphrase.sh #107

3 of 4 tasks complete
@DeviaVir

This comment has been minimized.

Show comment
Hide comment
@DeviaVir

DeviaVir Nov 1, 2017

Member

@drGrove do note that your commit is unverified until you update the public GPG in github.

Member

DeviaVir commented Nov 1, 2017

@drGrove do note that your commit is unverified until you update the public GPG in github.

@drGrove

This comment has been minimized.

Show comment
Hide comment
@drGrove

drGrove Nov 1, 2017

Member

It's because the email i'm using doesn't match my key. I need to add my other identities. I did this on my work computer and forgot to change the committer email -_-

Member

drGrove commented Nov 1, 2017

It's because the email i'm using doesn't match my key. I need to add my other identities. I did this on my work computer and forgot to change the committer email -_-

@drGrove

This comment has been minimized.

Show comment
Hide comment
@drGrove

drGrove Nov 2, 2017

Member

@DeviaVir @KellerFuchs commit signature is now verified

Member

drGrove commented Nov 2, 2017

@DeviaVir @KellerFuchs commit signature is now verified

@KellerFuchs

Same as hashbang/password-store#7 (review), you don't need to list the encryption subkey explicitely unless you have a particular reason to.

@KellerFuchs

This comment has been minimized.

Show comment
Hide comment
@KellerFuchs

KellerFuchs Nov 2, 2017

Member

@drGrove Like for hashbang/password-store#7, I pushed the changes I mentionned. Feel free to send a new PR if that's now what you need/want.

Member

KellerFuchs commented Nov 2, 2017

@drGrove Like for hashbang/password-store#7, I pushed the changes I mentionned. Feel free to send a new PR if that's now what you need/want.

@KellerFuchs KellerFuchs closed this Nov 2, 2017

@KellerFuchs KellerFuchs deleted the update-drgrove-key branch Nov 2, 2017

@KellerFuchs

This comment has been minimized.

Show comment
Hide comment
@KellerFuchs

KellerFuchs Nov 2, 2017

Member

Aaaah, my push was rejected for some silly reason 😭

Member

KellerFuchs commented Nov 2, 2017

Aaaah, my push was rejected for some silly reason 😭

@KellerFuchs KellerFuchs restored the update-drgrove-key branch Nov 2, 2017

@KellerFuchs KellerFuchs reopened this Nov 2, 2017

@KellerFuchs KellerFuchs merged commit a1f375b into master Nov 2, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@KellerFuchs KellerFuchs deleted the update-drgrove-key branch Nov 2, 2017

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