Skip to content

Conversation

@dimaglushkov
Copy link
Contributor

Hello and thanks for a great project!
I'm currently using this tool at my script to automatically get problem-related info to my local repo, create a code template for a solution, and automatically update a table with solved problems in README.md.
With this PR I've tried to fix two open issues

@fspv
Copy link
Owner

fspv commented Mar 5, 2022

Hi! Thanks for working on that! And so sorry for getting back to you so late. The recent events just knocked me down for a week and made me unable to work at all.

Anyway, your changes look good. Give me a second to verify them on my machine, and then I'll merge the repo and bump the version.

@fspv
Copy link
Owner

fspv commented Mar 5, 2022

Added one comment and tested both build and modified example.py. Everything LGTM as long as the comment is addressed.

Co-authored-by: Pavel Safronov <pv.safronov@gmail.com>
@dimaglushkov
Copy link
Contributor Author

Ah, that's must be some kind of a bad habit coz python isn't my main language :/
I've applied your suggestion and I believe that PR is safe to merge now
At this horrible time developing stuff is the only thing which keeps me from losing my mind
Anyway, stay safe

@fspv fspv merged commit 3d16ae0 into fspv:main Mar 6, 2022
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.

2 participants