Skip to content

Log - fix wrong types in docblock #128

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

Closed
wants to merge 2 commits into from
Closed

Conversation

keradus
Copy link
Collaborator

@keradus keradus commented Jan 9, 2018

No description provided.

@keradus
Copy link
Collaborator Author

keradus commented Jan 9, 2018

failure on 5.3/7+ was fixed on #122 and are not related to this PR.

@lyrixx, could you give a look on this PR and merge it please ?

@lyrixx
Copy link
Member

lyrixx commented Jan 10, 2018

Could you rebase ? Thanks.

@keradus
Copy link
Collaborator Author

keradus commented Jan 10, 2018

rebased

* @param int|null $offset start list from a given position
* @param int|null $limit limit number of fetched elements
* @param Repository $repository the repository where log occurs
* @param RevisionList|Revision|array $revisions a list of revisions or null if you want all history
Copy link
Member

@lyrixx lyrixx Jan 10, 2018

Choose a reason for hiding this comment

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

It could be also null? (default PHP value)

@lyrixx
Copy link
Member

lyrixx commented Jan 10, 2018

Good catch, thanks @keradus.

@lyrixx lyrixx closed this in 462d076 Jan 10, 2018
@keradus keradus deleted the types branch January 10, 2018 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants