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

Provide necessary write permissions to cache-cleaner #1813

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

Zeitsperre
Copy link
Collaborator

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes #xyz
  • Tests for the changes have been added (for bug fixes / features)
    • (If applicable) Documentation has been added / updated (for bug fixes / features)
  • CHANGES.rst has been updated (with summary of main changes)
    • Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

  • Provides the permissions for removing caches to the cache removal workflow

Does this PR introduce a breaking change?

No.

@Zeitsperre Zeitsperre added the bug Something isn't working label Jun 27, 2024
@Zeitsperre Zeitsperre self-assigned this Jun 27, 2024
@github-actions github-actions bot added the CI Automation and Contiunous Integration label Jun 27, 2024
Copy link
Contributor

@coxipi coxipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved!

@github-actions github-actions bot added the approved Approved for additional tests label Jun 27, 2024
@coveralls
Copy link

Coverage Status

coverage: 90.595%. remained the same
when pulling f891eb5 on fix-cache-cleaner
into e8d2f36 on main.

@Zeitsperre Zeitsperre merged commit 23fd681 into main Jun 27, 2024
28 of 29 checks passed
@Zeitsperre Zeitsperre deleted the fix-cache-cleaner branch June 27, 2024 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Approved for additional tests bug Something isn't working CI Automation and Contiunous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants