Skip to content
This repository has been archived by the owner on Nov 13, 2024. It is now read-only.

Can't delete tags with "#" (number sign) on it #10

Closed
AnimMouse opened this issue Apr 3, 2021 · 9 comments
Closed

Can't delete tags with "#" (number sign) on it #10

AnimMouse opened this issue Apr 3, 2021 · 9 comments

Comments

@AnimMouse
Copy link

I can't delete tags using this, I got this error:

Run dev-drprasad/delete-older-releases@v0.2.0
  with:
    keep_latest: 1
    delete_tags: true
    delete_tag_pattern: a
  env:
    GITHUB_TOKEN: ***
💬  no `repo` name given. fall-ing back to this repo
🔖  corresponding tags also will be deleted
releases containing a will be targeted
💬  found total of 2 matching active release(s)
🍻  found 1 older release(s)
starting to delete a#8 with id 39024378
🌶  failed to delete tag "a#8"  <- Reference does not exist
@dev-drprasad
Copy link
Owner

dev-drprasad commented Apr 3, 2021

Message implying that there is no tag exists with name "a#8". Can you cross whether that tag exists ? Ping me repo url if it is public, I will also check

@AnimMouse
Copy link
Author

@AnimMouse
Copy link
Author

The release was deleted, but the tag is not. Tested again on https://github.com/AnimMouse/ffmpeg-autobuild-gpl/runs/2260494888

@dev-drprasad
Copy link
Owner

If you don't mind can you release tag without "#" in it and delete it using this action. I just want to make sure "#" in tag is not the problem

@AnimMouse
Copy link
Author

Tried tags without "#" and it works, it seems like the "#" is the problem.

Run dev-drprasad/delete-older-releases@v0.2.0
💬  no `repo` name given. fall-ing back to this repo
🔖  corresponding tags also will be deleted
releases containing a will be targeted
💬  found total of 3 matching active release(s)
🍻  found 2 older release(s)
starting to delete a2 with id 40909959
starting to delete a1 with id 40909957
👍🏼  2 older release(s) deleted successfully!

@AnimMouse AnimMouse changed the title failed to delete tag "" <- Reference does not exist Can't delete tags with "#" (number sign) on it Jun 6, 2021
@thadguidry
Copy link
Collaborator

@AnimMouse we have a potential fix for this in #11 which would change to using regex for the delete_tag_pattern option. Do you think that might help your situation? Do you have strong feelings about my comment in that PR one way or another?

@AnimMouse
Copy link
Author

Yes, using regex is much better.

@yoomlam
Copy link

yoomlam commented Jun 1, 2023

+1 on using regex

@thadguidry
Copy link
Collaborator

Fixed in PR #30 and new release https://github.com/dev-drprasad/delete-older-releases/releases/tag/v0.3.1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants