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

#6 Added perms script #29

Closed
wants to merge 5 commits into from
Closed

Conversation

aryamagarwal
Copy link

@aryamagarwal aryamagarwal commented Mar 15, 2024

Fixes #6
I've written a Bash script to list the permissions of given files in octal format.
The script loops through each file to retrieve its permissions using the ls command and outputs the permissions in the specified format.

changes#6

@ecxtacy
Copy link

ecxtacy commented Mar 15, 2024

Kindly put proper PR heading.

@aryamagarwal aryamagarwal changed the title Adding Perms script. Solving #6 #6 Added perms script Mar 15, 2024
@aryamagarwal
Copy link
Author

Updated the PR heading.

scripts/perms Outdated Show resolved Hide resolved
@rootCircle
Copy link
Member

Please make the required changes!

You can use linter like shellcheck for your help!
To install use sudo apt install shellcheck & for use shellcheck file.sh

@rootCircle
Copy link
Member

ping me here when you're done with those, so we can move forward with your PR!

@rootCircle
Copy link
Member

Hey @aryamagarwal! Great work on fixing those issues, I am seeing that your code simply fails to work for directory! Can you check into that?

image

ping me here when you're done with those/need any help, so we can move forward with your PR!

@rootCircle
Copy link
Member

rootCircle commented Mar 16, 2024

LGTM 👍 Accepted! Great work @aryamagarwal! Thanks for your contribution!

also keep a good practice to write only relevant names for branches! like feat-perms fix-#6 etc

@rootCircle rootCircle requested review from rootCircle and removed request for rootCircle March 16, 2024 07:11
@rootCircle rootCircle closed this Mar 16, 2024
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.

Write perms script.
3 participants