Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement organization repository list support #28

Merged
merged 2 commits into from
Mar 20, 2016

Conversation

carols10cents
Copy link
Contributor

Hi! I added support for organization repository listing, in a similar way to how you did user repo listing. I'm more than happy to make any changes you'd like ❤️

@softprops
Copy link
Owner

Very nice. Thank you. I'm aiming to publish a release this weekend for all the new serde stuff. This will make a nice addition :)

@softprops
Copy link
Owner

Looks like travis didn't like that and the reason was ambiguous. I'll try to sort that out.

@softprops
Copy link
Owner

I think it may have to do with serde_codgen v0.7.1 ( the latest ) 0.7 release which as an upgraded dependency on syntex v0.30 ( from v0.29) while my cargo build specifies build dependencies on serde_codgen v0.7 and syntex v0.29. My guess is that cargo is pulling 0.7.1 and using syntex 0.29 which may, in someway cause this error.

@softprops
Copy link
Owner

@carols10cents can you try merging master and doing one more push to your branch. That should retrigger travis's test run against the different versions of rust again. I updated the my build dependency on syntex which I think should solve the problem

@carols10cents
Copy link
Contributor Author

Will do!!

@carols10cents
Copy link
Contributor Author

Looks like that worked!!

@carols10cents
Copy link
Contributor Author

I just added one more commit changing the doc comment i added-- I reread it and noticed it didn't make any sense, haha :) I fixed a few other little typos while I was in there too ❤️

softprops added a commit that referenced this pull request Mar 20, 2016
Implement organization repository list support
@softprops softprops merged commit 0dca45a into softprops:master Mar 20, 2016
@softprops
Copy link
Owner

published

@carols10cents carols10cents deleted the organizations branch March 23, 2016 00:03
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