Skip to content

Conversation

@jasonm
Copy link

@jasonm jasonm commented Feb 18, 2013

This pull fixes a small typo in the README.md, and also will serve as an example for the Molajo/Filesystem CLA at http://www.clahub.com/agreements/Molajo/Filesystem.

@jasonm
Copy link
Author

jasonm commented Feb 18, 2013

Hmm, only TravisCI seems to be updating the status here, I'm looking into why CLAHub isn't also updating the status.

@AmyStephen
Copy link
Member

@jasonm - you absolutely rock. You are providing a much needed, missing service.

I walked through the steps. That was helpful, so very easy to do, nice to download the data and have those signatures. Thanks very much.

Maybe I did something on the webhook? Let me take a look on my end.

@jasonm
Copy link
Author

jasonm commented Feb 18, 2013

A-ha! It looks like CLAHub did what it was supposed to that time:

Screen Shot 2013-02-17 at 11 50 06 PM

Thanks for the kind words @AmyStephen. So I think I have a good idea of why CLAHub was on the fritz that time.

It'll still be a little wonky until I work out clahub/clahub#27, with TravisCI and CLAHub both setting commit statuses, but you can at least look at the list of signees on clahub.com in the meantime.

@AmyStephen
Copy link
Member

Goodness, @jasonm - wow - that is really nice! That prevents any commits from occurring absent an agreement.

Cool. I'll keep an eye on #27. You let me know if having access will help.

Cheers!

@AmyStephen
Copy link
Member

BTW - the message is also in the status area for the issue report. So, it is very clear.

AND - it's right on the merge bar -- with a Merge with Caution note -- and the message right above.

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.

2 participants