Skip to content
This repository was archived by the owner on Oct 17, 2022. It is now read-only.

welcome #80

Merged
merged 5 commits into from
Oct 18, 2016
Merged

welcome #80

merged 5 commits into from
Oct 18, 2016

Conversation

stleon
Copy link
Contributor

@stleon stleon commented Oct 9, 2016

new version

new json
@nickva
Copy link
Contributor

nickva commented Oct 10, 2016

+1

I noticed trailing whitespace change on line 43 or so, it is currently preventing some Travis checks from passing. Let's fix that and I'll merge it

Thanks @stleon !

@stleon
Copy link
Contributor Author

stleon commented Oct 11, 2016

@nickva done

@nickva
Copy link
Contributor

nickva commented Oct 11, 2016

@stleon Thank you. Merged to master.

Verify it is on master and close the ticket when you get a a chance ( I rebased changes on top of latest master so I think it didn't auto-close on merge )

@stleon
Copy link
Contributor Author

stleon commented Oct 18, 2016

@nickva can you merge new changes?

@asfgit asfgit merged commit e97a665 into apache:master Oct 18, 2016
@nickva
Copy link
Contributor

nickva commented Oct 18, 2016

@stleon Done. Sorry for the delay. Thank you for your help!

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.

3 participants