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.

Sign up for GitHub

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

Jump to bottom

Unable to remove .ruff_cache folder #5543

Closed
1 task done
alexanderbazhenoff opened this issue Apr 18, 2024 · 3 comments · Fixed by #5548 or #5537
Closed
1 task done

Unable to remove .ruff_cache folder #5543

alexanderbazhenoff opened this issue Apr 18, 2024 · 3 comments · Fixed by #5548 or #5537
Labels
enhancement New feature or request O: backlog 🤖 Backlog, stale ignores this label

Comments

@alexanderbazhenoff
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Hi, again folder permissions when using standalone docker image in GitLab CI. This time .ruff_cache owned by root.

warning: failed to remove .ruff_cache/0.3.7/17270506990096333443: Permission denied
warning: failed to remove .ruff_cache/CACHEDIR.TAG: Permission denied
warning: failed to remove .ruff_cache/.gitignore: Permission denied

Expected Behavior

Why not to store all these temporary/cache files inside docker container by default? (instead of adding sudo rm -rf <new_path> to before_script on every version update :)

Anything else?

Thanks in advance for any ideas.
@alexanderbazhenoff alexanderbazhenoff added the enhancement New feature or request label Apr 18, 2024
@ferrarimarco ferrarimarco added the O: backlog 🤖 Backlog, stale ignores this label label Apr 18, 2024
ferrarimarco added a commit that referenced this issue Apr 19, 2024
@ferrarimarco
fix: configure ruff with a temp cache
12a2844 
- Configure Ruff to store its cache in a temporary directory inside the
  container by default. Users can still override this by providing a
  configuration file for Ruff.
- Add tests to ensure that super-linter deletes temporary files and
  directories.

Close #5543
@ferrarimarco ferrarimarco mentioned this issue Apr 19, 2024
Merged
8 tasks
@ferrarimarco
Copy link
Collaborator

#5548 should take care of this

@alexanderbazhenoff
Copy link
Author

@ferrarimarco thanks for the quick response!

@Okabe-Junya
Copy link
Contributor

Thank you for this PR as the contributor who opened #5456 ;) #5548 Looks Good to me!!

ferrarimarco added a commit that referenced this issue Apr 22, 2024
@ferrarimarco
fix: configure ruff with a temp cache (#5548)
56e675b 
- Configure Ruff to store its cache in a temporary directory inside the
  container by default. Users can still override this by providing a
  configuration file for Ruff.
- Add tests to ensure that super-linter deletes temporary files and
  directories.

Close #5543
@github-actions github-actions bot mentioned this issue Apr 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request O: backlog 🤖 Backlog, stale ignores this label
Projects
None yet
Milestone
No milestone
3 participants
@ferrarimarco @alexanderbazhenoff @Okabe-Junya