Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Update setup.py to load requirements from file and fix example in example.py #18

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

Merged
merged 3 commits into from
Mar 6, 2022
Merged

Update setup.py to load requirements from file and fix example in example.py #18

merged 3 commits into from
Mar 6, 2022

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