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

Adding documentation for the virtual printer plugin. #2142

Closed
wants to merge 1 commit into from

Conversation

pbackx
Copy link
Contributor

@pbackx pbackx commented Oct 4, 2017

What does this PR do and why is it necessary?

This PR is a pretty minor change: it adds some documentation on the virtual printer plugin. I was unable to locate much documentation and thought it would be useful for other developers such as me just getting started.

How was it tested? How can it be tested by the reviewer?

I ran make html in the docs folder and this ran without errors in the files. I did get a number of errors in other files which I will try to fix in a future PR.
I also inspected the resulting HTML file, which looks ok.

Any background context you want to provide?

It took me an evening to piece together the information on the virtual printer.

What are the relevant tickets if any?

None.

@foosel
Copy link
Member

foosel commented Oct 6, 2017

Thank you for this! I took the liberty to cherry-pick this to the maintenance branch (see 452ee8e) so that this will make it faster to the stable docs.

Since Github doesn't properly detect this I'll close this PR manually which will mark it as "closed" instead of "merged". Don't be alarmed by this, your PR was in fact merged.

@foosel foosel closed this Oct 6, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 28, 2020
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.

None yet

2 participants