Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Add CLA URL to error message#7310

Merged
ingorichter merged 1 commit intomasterfrom
jasonsanjose/cla-check-url
Mar 24, 2014
Merged

Add CLA URL to error message#7310
ingorichter merged 1 commit intomasterfrom
jasonsanjose/cla-check-url

Conversation

@jasonsanjose
Copy link
Copy Markdown
Member

Display CLA URL in travis log when the CLA check fails.

@jasonsanjose
Copy link
Copy Markdown
Member Author

Decided that the travis log details (e.g. from this PR #7295) wasn't helpful enough for new contributors to resolve the build failure on their own.

@ingorichter ingorichter self-assigned this Mar 24, 2014
@ingorichter
Copy link
Copy Markdown
Contributor

This is a good idea. Thanks for adding.

ingorichter added a commit that referenced this pull request Mar 24, 2014
@ingorichter ingorichter merged commit 2a9d945 into master Mar 24, 2014
@TomMalbran TomMalbran deleted the jasonsanjose/cla-check-url branch March 24, 2014 20:48
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