Skip to content
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

Added own pattern for similar file types. #96

Closed
wants to merge 2 commits into from

Conversation

shuklamaneesh23
Copy link

Solved issue #7 ,
This pull request extends the functionality of the file organization script by adding support for additional file types. The script now categorizes files into folders based on their extensions, including common formats such as PDFs, images, videos, music, web files, scripting files, text files, documents, configuration files, and compressed files.

Expanded the existing configuration to include additional file types such as PDFs, images (PNG, JPG, JPEG, GIF, SVG), videos (MP4, AVI, MKV, MOV), music (MP3, WAV, ALAC), web files (HTML), scripting files (JS, SH), text files (TXT), documents (DOC, DOCX), configuration files (JSON, YAML, YML), and compressed files (ZIP, GZ, BZ2).

@rootCircle
Copy link
Member

hi @shuklamaneesh23, thanks for your PR! few problems

mkdir: cannot create directory ‘/home/user/FOSS/bash-practice-repo-24/scripts/rm-bins’: File exists

Remove warning if directory already exists!

egrep: warning: egrep is obsolescent; using grep -E
grep: warning: * at start of expression

egrep is obsolescent; replace its usage with grep -E

@shuklamaneesh23
Copy link
Author

@rootCircle , fixed the above mentioned issues now.please,review the latest commit and let me know if any corrections required.

@rootCircle
Copy link
Member

grep: warning: * at start of expression

mkdir: cannot create directory ‘/home/user/FOSS/bash-practice-repo-24/main’: File exists

mv: 'main' and '/home/user/FOSS/bash-practice-repo-24/main' are the same file

There are some more warning in your implementation! since the author provided the basic working prototype based on the issue, marks will be provided based on that!

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

Successfully merging this pull request may close these issues.

2 participants