Skip to content

BUILD-8291 add pr_cleanup action#6

Merged
tomverin merged 2 commits intomasterfrom
feat/tom/BUILD-8291
Jun 27, 2025
Merged

BUILD-8291 add pr_cleanup action#6
tomverin merged 2 commits intomasterfrom
feat/tom/BUILD-8291

Conversation

@tomverin
Copy link
Copy Markdown
Contributor

No description provided.

@tomverin tomverin force-pushed the feat/tom/BUILD-8291 branch 2 times, most recently from 8c32a44 to f582b22 Compare June 27, 2025 09:49
@tomverin tomverin marked this pull request as ready for review June 27, 2025 09:50
@tomverin tomverin requested a review from a team June 27, 2025 09:50
Comment thread pr_cleanup/action.yml Outdated
Comment thread pr_cleanup/cleanup.sh
Comment thread pr_cleanup/cleanup.sh
@tomverin tomverin requested a review from Copilot June 27, 2025 12:20
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new GitHub Action to automatically clean up caches and artifacts associated with pull requests. Key changes include:

  • A Bash script (cleanup.sh) for listing and deleting caches and artifacts.
  • A composite GitHub Action definition (action.yml) to execute the cleanup script.
  • Updated documentation in README.md outlining usage and inputs.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
pr_cleanup/cleanup.sh Adds a Bash script to fetch and delete caches and artifacts.
pr_cleanup/action.yml Defines the PR Cleanup action and sets up its environment.
README.md Documents the new pr_cleanup action and its usage.
Comments suppressed due to low confidence (1)

pr_cleanup/action.yml:24

  • Consider adding an inline comment explaining the purpose and expected behavior of the GH_FORCE_TTY variable, as the use of the magic number 140 might not be immediately clear to future maintainers.
        GH_FORCE_TTY: 140

Comment thread pr_cleanup/cleanup.sh
@tomverin tomverin force-pushed the feat/tom/BUILD-8291 branch 3 times, most recently from 2fa21d5 to 69716f7 Compare June 27, 2025 13:24
Comment thread pr_cleanup/action.yml Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you think that you can add a test workflow creating a cache, an artifact, and calling the action for cleanup?

@tomverin
Copy link
Copy Markdown
Contributor Author

Do you think that you can add a test workflow creating a cache, an artifact, and calling the action for cleanup?

let me try this

@tomverin tomverin force-pushed the feat/tom/BUILD-8291 branch from 69716f7 to 53ce6b5 Compare June 27, 2025 13:28
@tomverin tomverin requested a review from matemoln June 27, 2025 14:07
@tomverin tomverin force-pushed the feat/tom/BUILD-8291 branch from 1d579b5 to d9acc26 Compare June 27, 2025 14:09
@tomverin tomverin dismissed matemoln’s stale review June 27, 2025 14:18

all the changes were included

@tomverin tomverin force-pushed the feat/tom/BUILD-8291 branch from d9acc26 to 9553df1 Compare June 27, 2025 14:20
@tomverin tomverin enabled auto-merge (squash) June 27, 2025 14:21
@sonarqubecloud
Copy link
Copy Markdown

@tomverin tomverin merged commit a3e0a6c into master Jun 27, 2025
6 of 7 checks passed
@tomverin tomverin deleted the feat/tom/BUILD-8291 branch June 27, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants