inspirezonetech / TeachMePythonLikeIm5

Teach the Python programming language using a collection of super beginner friendly tutorials and challenges.
https://inspirezone.tech/
MIT License
25 stars 43 forks source link

Populate file-handling/scanning-files.py with a tutorial and challenge. #101

Closed CelestialZeus closed 2 years ago

CelestialZeus commented 2 years ago

Please fill in this pull request template before submitting

1. This pull request resolves : https://github.com/inspirezonetech/TeachMePythonLikeIm5/issues/18

2. Description

Populate file-handling/scanning-files.py with a tutorial and challenge.

3. Fill in checklist by marking [x]

funbeedev commented 2 years ago

@MalevolentStrix Thanks for your PR!

Just one part to check with you. When I uncomment this:

filename = input("Enter the directory name/path you would like to scan: ")  # uncomment if you want to give the directory name while running

I cannot seem to get the program to work and it always prints the directory not found error. Can you please explain what format the input is expecting the directory to be entered? Or can check your program is running correctly before I can merge? Thanks!

CelestialZeus commented 2 years ago

@MalevolentStrix Thanks for your PR!

Just one part to check with you. When I uncomment this:

filename = input("Enter the directory name/path you would like to scan: ")  # uncomment if you want to give the directory name while running

I cannot seem to get the program to work and it always prints the directory not found error. Can you please explain what format the input is expecting the directory to be entered? Or can check your program is running correctly before I can merge? Thanks!

So sorry for the mistake, I actually misspelled a variable. Can you check it once more, I have pushed it and now it gives me the correct output, you can write "./" for scanning the files inside the current directory or specify the full location to scan files there.

funbeedev commented 2 years ago

@MalevolentStrix Thanks for fixing the issue. Happy to merge now :smile: