-
Notifications
You must be signed in to change notification settings - Fork 60
extract files from zip in python #363
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
Conversation
@@ -0,0 +1,14 @@ | |||
#import pyshorteners library |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you explain to me the use of this file here? @Sapna2001
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I extracted this script from a zip file to show an example. I will remove it.
@@ -0,0 +1,14 @@ | |||
#import pyshorteners library | |||
import pyshorteners | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you need to remove this URL shortener script from this pr.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok
# Enter the name of zip file | ||
zipFile = input("Enter zip file:") | ||
# Enter the path of the file to be extracted | ||
extractFile = input("Enter path of the file:") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can take any one input from user. Either file name or path. are both really required?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have made the changes.
Your checklist for this pull request
Parent issue: #347
Type of change
Select all that applies:
Checklist:
❤️ Thank you!