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

Add support for querying the CheckUser log #86

Merged
merged 2 commits into from Jun 20, 2015

Conversation

lfaraone
Copy link
Contributor

No description provided.

@danmichaelo
Copy link
Member

Do you know the reason for the extra level? At first sight it seems kinda redundant.

Anyways, would you mind fix the style issues? (Please amend them into the last commit and force push) https://travis-ci.org/mwclient/mwclient/jobs/66779057#L387

And thanks for noticing the print statement ;)

@lfaraone
Copy link
Contributor Author

Do you know the reason for the extra level? At first sight it seems kinda redundant.

Nope; that's just how the API works, unfortunately. 😦 The MW API is hardly a paragon of consistency.

Anyways, would you mind fix the style issues?

Done; amended and repushed.

@waldyrious
Copy link
Member

I'll assume from @danmichaelo's previous comment that there aren't other issues with this PR and merge. Thanks!

waldyrious added a commit that referenced this pull request Jun 20, 2015
Add support for querying the CheckUser log
@waldyrious waldyrious merged commit 67a3bad into mwclient:master Jun 20, 2015
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.

None yet

3 participants