这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on Aug 23, 2024. It is now read-only.

#122 Added pagination support for repo.commits. #145

Merged
merged 3 commits into from
Oct 7, 2014

Conversation

kunzimariano
Copy link
Contributor

No description provided.

@@ -38,8 +38,8 @@ class Repo

# Get the commits for a repository
# '/repos/pksunkara/hub/commits' GET
Copy link
Owner

Choose a reason for hiding this comment

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

Could you add the comments here?

   # - page or query object, optional - params[0]
   # - per_page, optional             - params[1]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure thing.

@@ -1,4 +1,4 @@
// Generated by CoffeeScript 1.7.1
// Generated by CoffeeScript 1.8.0
Copy link
Owner

Choose a reason for hiding this comment

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

Oh no. I haven't updated the library to 1.8 yet, please use 1.7.1 to generate the lib.

@pksunkara
Copy link
Owner

Sorry about this, but just a few minor nitpicks.

@kunzimariano
Copy link
Contributor Author

No worries, let me downgrade coffee script so I can re-generate the files.

- Added white space after params...
- Re-generated js files with CoffeeScript 1.7.1
pksunkara added a commit that referenced this pull request Oct 7, 2014
#122 Added pagination support for repo.commits.
@pksunkara pksunkara merged commit bf9965c into pksunkara:master Oct 7, 2014
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