Skip to content

Conversation

@chaydont
Copy link
Contributor

@chaydont chaydont commented Oct 6, 2020

ch55059
Need an instance with at least dip#11418 merged (not on master yet)

@chaydont chaydont self-assigned this Oct 6, 2020
@chaydont chaydont requested a review from FChataigner November 19, 2020 10:40
@chaydont chaydont added this to the 9.0 milestone Nov 19, 2020
Copy link
Contributor

@FChataigner FChataigner left a comment

Choose a reason for hiding this comment

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

looks good

"""
Download the whole wiki of the project in PDF format into the given output file.
"""
stream = self.get_export_stream(paperSize=paperSize, exportAttachment=exportAttachment)
Copy link
Contributor

Choose a reason for hiding this comment

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

is it possible to use with ... as stream: instead of the get/do/close construction? (if the stream can handle it)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added it, it's indeed neater. Also I used underscores instead of camelCase for parameter names

@chaydont chaydont requested a review from FChataigner November 26, 2020 15:24
@chaydont chaydont merged commit f639844 into master Dec 3, 2020
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.

3 participants