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

scripts/pdf/render.py: fix typo #5206

Merged
merged 1 commit into from
Jan 31, 2021
Merged

Conversation

bl-ue
Copy link
Contributor

@bl-ue bl-ue commented Jan 30, 2021

tdlr-pages-to-PDF β†’ tldr-pages-to-pdf πŸ˜„

@bl-ue bl-ue added the tooling Helper tools, scripts and automated processes. label Jan 30, 2021
@bl-ue bl-ue requested a review from navarroaxel January 30, 2021 21:39
@bl-ue
Copy link
Contributor Author

bl-ue commented Jan 30, 2021

When merging, please correct the commit message if necessary, I forgot to change it before I commited...

@bl-ue
Copy link
Contributor Author

bl-ue commented Jan 30, 2021

I plan on reworking that script sometime. I'm a Python expert and that file could use some more cleanup πŸ˜‰

@bl-ue bl-ue changed the title scripts/pdf/render.py: fix tldr scripts/pdf/render.py: fix typo Jan 30, 2021
Copy link
Member

@sbrl sbrl left a comment

Choose a reason for hiding this comment

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

Thanks!

Yeah, some rcleanup would be appreciated. I don't think was the one who wrote it (but if I did I don't remember), but my Python is rather rusty due to a preference for bracketed languages like JS and C# - so some assistance here would definitely be welcome.

@sbrl sbrl merged commit dc690e3 into master Jan 31, 2021
@sbrl sbrl deleted the scripts-pdf-render.py-fix-typo branch January 31, 2021 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tooling Helper tools, scripts and automated processes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants