Skip to content

Conversation

@sdbedi
Copy link

@sdbedi sdbedi commented Jan 3, 2018

Checklist
  • documentation is added or updated

@openssl-machine openssl-machine added the hold: cla required The contributor needs to submit a license agreement label Jan 3, 2018
@mattcaswell
Copy link
Member

This appears to be a trivial change. If you agree please add the text "CLA: trivial" to your commit description to clear the "need-cla" label.

@bernd-edlinger
Copy link
Member

you can use "git commit --amend" to do that.

Copy link
Member

@levitte levitte left a comment

Choose a reason for hiding this comment

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

Formality: this is a legally trivial change

@richsalz
Copy link
Contributor

richsalz commented Jan 7, 2018

If @sdbedi won't agree, then we have to close this and resubmit ourselves.
-1

@mspncp
Copy link
Contributor

mspncp commented Jan 7, 2018

Note that this single line change is also part of #5027, so you could just close this PR and continue discussion there.

@richsalz
Copy link
Contributor

see the other referenced issues; closing this as a dup. Thank you for bringing this to our attention!

@richsalz richsalz closed this Jan 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold: cla required The contributor needs to submit a license agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants