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

prints permission in octal format #6 #82

Closed
wants to merge 2 commits into from

Conversation

AMANVISHWAKARMA27
Copy link

Fixes #6
image

@AMANVISHWAKARMA27 AMANVISHWAKARMA27 changed the title prints permission in octal format prints permission in octal format #6 Mar 16, 2024
@rootCircle
Copy link
Member

hi @AMANVISHWAKARMA27, thanks for contribution!

few things, please include function usage in the code itself, include screenshot in PR description as well!

@rootCircle
Copy link
Member

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

@AMANVISHWAKARMA27
Copy link
Author

AMANVISHWAKARMA27 commented Mar 17, 2024

hi @AMANVISHWAKARMA27, thanks for contribution!

few things, please include function usage in the code itself, include screenshot in PR description as well!

@rootCircle, if u could explain me in a bit more detail what I have to do ?

@rootCircle
Copy link
Member

@rootCircle, if u could explain me in a bit more detail what I have to do ?

currently you have the function, but we are not using up to satisfy the issue description!

simply put, issue description requires something like this given below, but your code does doesn't provide the usage

> perms README.md
642 README.md

@AMANVISHWAKARMA27
Copy link
Author

@rootCircle, if u could explain me in a bit more detail what I have to do ?

currently you have the function, but we are not using up to satisfy the issue description!

simply put, issue description requires something like this given below, but your code does doesn't provide the usage

> perms README.md
642 README.md

Ahh got it. Doing it asap.

@AMANVISHWAKARMA27
Copy link
Author

@rootCircle , it is done now as per your suggestion.
Screenshot 2024-03-17 105932

@rootCircle
Copy link
Member

LGTM 👍 Accepted!

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.
2 participants