-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fixes runme when it attempts to access unreadable files or directories #646
Fixes runme when it attempts to access unreadable files or directories #646
Conversation
5a5535d
to
097e13f
Compare
d5e3662
to
81da0e8
Compare
81da0e8
to
8c75900
Compare
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.
Some questions.
2897caa
to
70e97f1
Compare
bca13ad
to
399c008
Compare
399c008
to
3e518a1
Compare
@pastuxso is this still in draft since it's gotten two reviews already? |
@sourishkrout ready for review. I'll push in a separate branch an attempt to improve test running using a non-root user within Docker. |
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.
almost there
This reverts commit 169095f.
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.
✅ LGTM
Quality Gate passedIssues Measures |
Handling the os.ErrPermission when an error occurs while trying to read files without proper permissions.
Closes: #645