Skip to content

Conversation

@jayaganeshkumar
Copy link
Contributor

Pull Request Template

script name - zip_extractor

What have you Changed

what you changed in the codebase.write here

Created a folder with zip extractor python script along with requirements.txt and readme.md files

Issue no.(must) - #33

Self Check(Tick After Making pull Request)

  • This issue was assigned to me.
  • One Change in one Pull Request
  • My file is in proper folder (Name of folder should be in lowercase with no space in between)
  • I am following clean code and Documentation and my code is well linted with flake8.
  • I have added README.md and requirements.txt with my script

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

Copy link
Contributor

@pawangeek pawangeek left a comment

Choose a reason for hiding this comment

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

Correct requirements.txt

@jayaganeshkumar
Copy link
Contributor Author

@pawangeek what I have to do?

@@ -0,0 +1,2 @@
zipfile37
Copy link
Contributor

Choose a reason for hiding this comment

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

Please correct this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is the library name. Can I leave the library names as just "zipfile"

Copy link
Contributor

Choose a reason for hiding this comment

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

But what about the version. You can use "pip freeze > requiements.txt" to check that

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 don't know that feature. If you know that can you update the readme file.

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, that's an inbuilt library. My bad

@pawangeek pawangeek linked an issue Oct 9, 2020 that may be closed by this pull request
2 tasks
@pawangeek pawangeek merged commit 9acc9a6 into python-geeks:main Oct 9, 2020
@pawangeek pawangeek added the hacktoberfest-accepted Supporting completion of hacktober fest label Oct 9, 2020
@jayaganeshkumar
Copy link
Contributor Author

Thank you so much @pawangeek

@jayaganeshkumar
Copy link
Contributor Author

Can I have your linkedin profile id so that we can talk related to these kind of stuff?

@pawangeek
Copy link
Contributor

You can use @pkjains to ping me on telegram. LinkedIn is not suitable for this kind of stuff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted Supporting completion of hacktober fest

2 participants