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

Preserve existing repo topic tags when adding "nextstrain" and "pathogen" #10

Merged
merged 1 commit into from
May 24, 2024

Conversation

tsibley
Copy link
Member

@tsibley tsibley commented May 24, 2024

I realized before deploying "Tag each pathogen repo with the "nextstrain" and "pathogen" GitHub topics" (a80ad9a) that it would replace all topics instead of amending/extending them.

Additionally, note the necessary fine-grained permission to set repo topics in the README. The lack of this permission on my admin token was what led to me to realizing that topics are always set en masse instead of added/removed individually, as I looked up the permission necessary on the API endpoint and noticed its documented behaviour.

Related-to: #9

Checklist

  • Checks pass

…gen"

I realized before deploying "Tag each pathogen repo with the
"nextstrain" and "pathogen" GitHub topics" (a80ad9a) that it would
_replace_ all topics instead of amending/extending them.

Additionally, note the necessary fine-grained permission to set repo
topics in the README.  The lack of this permission on my admin token was
what led to me to realizing that topics are always set en masse instead
of added/removed individually, as I looked up the permission necessary
on the API endpoint and noticed its documented behaviour.

Related-to: <#9>
@tsibley tsibley merged commit 63b7d05 into main May 24, 2024
1 check passed
@tsibley tsibley deleted the trs/github-repo-topics-non-exhaustive branch May 24, 2024 17:43
@tsibley tsibley requested a review from a team May 24, 2024 17:44
README.md Show resolved Hide resolved
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