search

Reviewable / appengine-remove-action

Action to remove versions of deployed App Engine apps over the set limit
3 stars 2 forks source link

feature request: remove specific version #1

Open maapteh opened 3 years ago

maapteh commented 3 years ago

Im using this action perfectly. But for closing of pr versions we know the version. Is it possible to also pass a specific version and then just remove that version?

And maybe instead of a max number, also an optional max of days :) haha, if i understand how to run it locally i can propose a PR?

pkaminski commented 3 years ago

I'd accept a PR that takes an optional number of days. But for removing a specific version, I think it would be better for you to make a separate action -- the intent is distinct, and the API will be completely different. Feel free to fork!