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

write megalinter-reports as current user instead of root #2594

Closed
gbrandt1 opened this issue Apr 24, 2023 · 3 comments
Closed

write megalinter-reports as current user instead of root #2594

gbrandt1 opened this issue Apr 24, 2023 · 3 comments
Labels
enhancement New feature or request O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity

Comments

@gbrandt1
Copy link

i think it's quite annoying and unneccessary that megalinter-reports/ gets written as the root user.
especially for users that don't have sudo rights to fix it.
i guess this comes from running the docker container as root and ignoring the current user.

it would be better to either run as the current user, or change the owner of megalinter-reports to the current user by default.

a --user option should be added to mega-linter-runner.

cheers,
gerhard

@gbrandt1 gbrandt1 added the enhancement New feature or request label Apr 24, 2023
@nvuillam
Copy link
Member

Related to #1985
cc @Kurt-von-Laven

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity label May 25, 2023
@Kurt-von-Laven
Copy link
Collaborator

Kurt-von-Laven commented May 25, 2023

Duplicate #1975. I recommend using rootless Docker in the meantime. See ScribeMD/rootless-docker for a GitHub Actions that does this. Alternatively, on beta you can set REPORT_OUTPUT_FOLDER: none in your MegaLinter config to disable generation of the report folder entirely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request O: stale 🤖 This issue or pull request is stale, it will be closed if there is no activity
Projects
None yet
Development

No branches or pull requests

3 participants