Repository navigation
Cite MycoBank's current web services among the list's sources - #205
Merged
Merged
Conversation
The infectious-agent list's README cited MycoBank's web services at /westerdijk/public, which answers 503. The list's reconciliation read MycoBank's records through /westerdijk/mycobank, the API MycoBank's curators point to; its base needs a login and has no page of its own, so the README names it as code rather than linking it. The ICTV link drops its trailing slash, which redirected. The list's changelog records the fix, since the README ships in the list's release archive.
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues remain for the documentation-only source-reference updates.
No open issues 🎉
What changed in this PR
This documentation-only PR updates source references for the infectious-agent list. The list’s content is unchanged.
Changes:
- Names MycoBank’s authenticated web-service endpoint as inline code and removes the trailing slash from the ICTV taxonomy link.
- Records the documentation fixes under Unreleased because the README ships in the release archive.
| File | Description |
|---|---|
metadata/common/infectious-agents/README.md |
Updates the MycoBank endpoint and ICTV taxonomy link. |
metadata/common/infectious-agents/CHANGELOG.md |
Records the source-reference fixes under Unreleased. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The infectious-agent list's README cited MycoBank's web services at
https://webservices.bio-aware.com/westerdijk/public, which answers 503. The list's reconciliation with MycoBank read its records throughhttps://webservices.bio-aware.com/westerdijk/mycobank, the API MycoBank's curators point to. That address needs a login and has no page of its own, so the README now names it as code rather than linking it, which also keeps the link check from flagging it.The ICTV taxonomy link drops its trailing slash, from which it redirected, and points at
https://ictv.global/taxonomy.The README is bundled into the list's release archive, so the list's changelog records the fix under Unreleased. The list's content is unchanged.