Skip to content

Conversation

jochenberger
Copy link
Contributor

@p0deje
Copy link
Member

p0deje commented Jul 13, 2017

Have you signed CLA?

@jochenberger
Copy link
Contributor Author

No. Is that really necessary for a simple typo fix?

@p0deje
Copy link
Member

p0deje commented Jul 13, 2017

@jochenberger
Copy link
Contributor Author

Never mind then.

@ddavison ddavison reopened this Jul 13, 2017
@ddavison
Copy link
Member

ddavison commented Jul 13, 2017

We go by the Obvious Fix Rule

HOWEVER, very small contributions such as fixing spelling errors, where the content is small enough to not be considered intellectual property, can be submitted by a contributor as a patch, without a [CLA] sign-off.

In order to not discourage PRs from the community, this should be merged in. 👍

@shs96c
Copy link
Member

shs96c commented Jul 16, 2017

Indeed. The Obvious Fix Rule clearly applies here.

@shs96c shs96c merged commit 355f7f2 into SeleniumHQ:master Jul 16, 2017
@p0deje
Copy link
Member

p0deje commented Jul 25, 2017

Thank you, I wasn't even aware of "Obvious Fix Rule".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants