Skip to content

Conversation

@christopher-dG
Copy link
Member

@DilumAluthge
Copy link
Member

Why not just create annotated tags by default?

@christopher-dG
Copy link
Member Author

Good question, I never really thought that of that to be honest. I guess one thing I like is that when you create a lightweight tag and the referenced commit is signed, you get the nice green "Verified" next to the release too. But that's a pretty minor thing, I think only doing annotated tags would be good as well.

@DilumAluthge
Copy link
Member

DilumAluthge commented Mar 5, 2020

I guess the question is: suppose you use the GitHub web interface to create a new GitHub release for which the tag does not exist. GitHub will automatically create a new tag. What kind of tag does it create?

We should create the same kind of tag.


I don't know the answer to this question. I guess one of us has to make a dummy repo and test it out.

@christopher-dG
Copy link
Member Author

It's lightweight.

@christopher-dG
Copy link
Member Author

Thinking a bit more about it, I think it probably makes the most sense to just always make annotated tags. I like the consistency of always creating the same type of tag. And there are times where we need to annotate tags (GPG) as well as people who expect the tags to be annotated (the Discourse poster).

@DilumAluthge
Copy link
Member

Thinking a bit more about it, I think it probably makes the most sense to just always make annotated tags. I like the consistency of always creating the same type of tag. And there are times where we need to annotate tags (GPG) as well as people who expect the tags to be annotated (the Discourse poster).

I guess that's probably the simplest approach. Always do annotated and there is no confusion.

@DilumAluthge
Copy link
Member

@christopher-dG Update the title of this PR?

@christopher-dG christopher-dG changed the title Add option to force annotated tags Always make annotated tags Mar 7, 2020
@christopher-dG christopher-dG merged commit 1e7f2fe into master Mar 7, 2020
@christopher-dG christopher-dG deleted the cdg/annotate branch March 7, 2020 05:12
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.

3 participants