Skip to content
This repository has been archived by the owner on Sep 25, 2020. It is now read-only.

Implemented schemas.video.category #8

Merged

Conversation

BenjaminSchaaf
Copy link
Contributor

Added schemas.video.category with spec


# Public: Get video categories.
#
# See: http://open.youku.com/docs?id=90
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ curl -vvv http://open.youku.com/docs?id=90
< HTTP/1.1 301 Moved Permanently
...
< Location: http://cloud.youku.com?id=90

Can you change to http://cloud.youku.com/docs?id=90

@avokhmin
Copy link
Contributor

@BenjaminSchaaf can you update README.md and CHANGELOG.md too?

@BenjaminSchaaf
Copy link
Contributor Author

I updated/cleaned up the readme, but there is no changelog currently. Is it really worth maintaining a changelog when we have git log?

@avokhmin
Copy link
Contributor

I updated/cleaned up the readme, but there is no changelog currently. Is it really worth maintaining a changelog when we have git log?

Why not, it is not hard but very useful. See: https://github.com/Shuttlerock/youku_client/releases

@BenjaminSchaaf
Copy link
Contributor Author

Exactly, we already keep descriptions of high level changes in release tags, so why duplicate them in a changelog which can't always be fast forwarded?

@avokhmin
Copy link
Contributor

I try to use best practices, see: https://github.com/plataformatec/devise
They have CHANGELOG.md and the same info in https://github.com/plataformatec/devise/releases

@BenjaminSchaaf
Copy link
Contributor Author

ok, I've added a Unreleased section with this pull request

@avokhmin
Copy link
Contributor

👍

avokhmin added a commit that referenced this pull request Dec 10, 2015
@avokhmin avokhmin merged commit 29ee17f into Shuttlerock-Archive:master Dec 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants