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 prefix: dpvocab #1240

Merged
merged 9 commits into from
Nov 2, 2024
Merged

Add prefix: dpvocab #1240

merged 9 commits into from
Nov 2, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 31, 2024

Closes #1241

@github-actions github-actions bot added New Used in combination with prefix, metaprefix, or collection for new entries Prefix labels Oct 31, 2024
@github-actions github-actions bot changed the title Add prefix: gtdb Add prefix: dpvocab Oct 31, 2024
@github-actions github-actions bot force-pushed the create-pull-request/patch-21ea943 branch from 33e2274 to 963e5b5 Compare October 31, 2024 18:49
Comment on lines 28721 to 28723
"publications": [
{}
],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"publications": [
{}
],
"publications": [
{
"doi": "10.48550/arXiv.2404.13426"
}
],

Comment on lines 28709 to 28715
"comment": "The resource itself specifies a preferred prefix of `dpv` but this is already claimed in Bioregistry by the Description of Plant Viruses (https://bioregistry.io/registry/dpv)",
"contributor": {
"email": "[email protected]",
"github": "caufieldjh",
"name": "Harry Caufield",
"orcid": "0000-0001-5705-7831"
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"comment": "The resource itself specifies a preferred prefix of `dpv` but this is already claimed in Bioregistry by the Description of Plant Viruses (https://bioregistry.io/registry/dpv)",
"contributor": {
"email": "[email protected]",
"github": "caufieldjh",
"name": "Harry Caufield",
"orcid": "0000-0001-5705-7831"
},
"comment": "The resource itself specifies a preferred prefix of `dpv` but this is already claimed in Bioregistry by the Description of Plant Viruses (https://bioregistry.io/registry/dpv)",
"contact": {
"email": "[email protected]",
"github": "coolharsh55",
"name": "Harshvardhan J. Pandit",
"orcid": "0000-0002-5068-3714"
},
"contributor": {
"email": "[email protected]",
"github": "caufieldjh",
"name": "Harry Caufield",
"orcid": "0000-0001-5705-7831"
},

@bgyori
Copy link
Contributor

bgyori commented Nov 1, 2024

Thanks @caufieldjh! I made a bunch of suggestions but there seems to be a glitch in GitHub and I can't apply them through the UI here.

@github-actions github-actions bot changed the title Add prefix: dpvocab Add prefix: gtdb Nov 1, 2024
@github-actions github-actions bot force-pushed the create-pull-request/patch-21ea943 branch from 963e5b5 to c6717c3 Compare November 1, 2024 16:22
@bgyori
Copy link
Contributor

bgyori commented Nov 1, 2024

Oh no, the Github action is buggy and force pushed the wrong content to this branch - we'll work on reconstructing it.

@bgyori bgyori changed the title Add prefix: gtdb Add prefix: dpvocab Nov 1, 2024
@nagutm
Copy link
Collaborator

nagutm commented Nov 1, 2024

Oh no, the Github action is buggy and force pushed the wrong content to this branch - we'll work on reconstructing it.

See #1244

nagutm and others added 6 commits November 2, 2024 08:56
This PR fixes
#1240 (comment)
which was caused by a buggy github-actions that forcefully overwrote the
content of the `dpvocab` prefix with another prefix.

Co-authored-by: Mufaddal Naguthanawala <[email protected]>
Co-Authored-By: Benjamin M. Gyori <[email protected]>
Co-Authored-By: Benjamin M. Gyori <[email protected]>
Co-Authored-By: Mufaddal Naguthanawala <[email protected]>
@cthoyt cthoyt merged commit d9a9df5 into main Nov 2, 2024
13 checks passed
@cthoyt cthoyt deleted the create-pull-request/patch-21ea943 branch November 2, 2024 13:29
bgyori added a commit that referenced this pull request Nov 19, 2024
The GitHub Actions workflow for registering new prefixes through the
issue template is reusing the same branch for each new issue, leading to
the content of the second pull request overwriting the first. This was
first noticed in this PR:
#1240. This occurs
because the workflow does not generate a unique branch for each new
issue. As a result, subsequent PRs are force-pushing content to the
previous PR's branch instead of creating new, independent branches for
each PR.

This PR introduces the following changes to the `new_prefix_pr.yml` to
address this issue:
- Create a step to extract the issue number from the issue URL.
- Dynamically generate a unique branch name
(`create-pull-request/patch-<issue_number>`) for each new issue.
- Modify the PR creation step to use the dynamically generated branch
name, ensuring that each PR is linked to its own unique branch.

---------

Co-authored-by: Mufaddal Naguthanawala <[email protected]>
Co-authored-by: Benjamin M. Gyori <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Used in combination with prefix, metaprefix, or collection for new entries Prefix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add prefix dpvocab
4 participants