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

Add dry-run param #119

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add dry-run param #119

wants to merge 1 commit into from

Conversation

vpipkt
Copy link

@vpipkt vpipkt commented Nov 22, 2023

Logs the ids of the artifacts that would be deleted instead of deleting them. This should allows folks to test out what would happen before taking permanent action.

Closes #117

@vpipkt vpipkt requested a review from a team as a code owner November 22, 2023 12:51
@vpipkt
Copy link
Author

vpipkt commented Nov 29, 2023

Hi @actions/packages can anyone take a look at this?

@AtomicFS
Copy link

Hi, I see there are 2 of these pull requests (#9 ) but none of the was ever merged. I would love to have this feature. Any chance of getting either of these revived and merged?

@AtomicFS AtomicFS mentioned this pull request Feb 21, 2024
@ivan-penchev
Copy link

@jidicula @kgrzebien @takost @s-anupam @nishthaGupta could you please kindly look into this, it is a very reasonable feature to have.

@javs-perez
Copy link

Hi all, here from the Packages team at GitHub.

Thank you for opening this request! It's on our radar and has been added to our list of items to tackle. While we don't have a specific timeline for when it will be reviewed, please know we're on it and appreciate your patience and support.

@Drowze
Copy link

Drowze commented Jul 30, 2024

Looking forward to this :)

Would also be appreciated to set an output with the list of deleted versions (or perhaps append that information to a step summary)

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.

Add dry run parameter
5 participants