Skip to content

Clarify ICTV and EVORAO guidance for virology - #1874

Merged
bianchini88 merged 4 commits into
elixir-europe:masterfrom
EVORA-project:docs/clarify-virology-ontologies
Nov 6, 2026
Merged

bianchini88 merged 4 commits into
elixir-europe:masterfrom
EVORA-project:docs/clarify-virology-ontologies

Conversation

@Angatar

@Angatar Angatar commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Clarifies the complementary roles of ICTV, the ICTV Ontology and API, and EVORAO on the virology page.

@github-actions

Copy link
Copy Markdown
Contributor

Before approving and merging a pull request (PR), the editors must check that

  • The page layout in the preview looks correct.
  • The new page is linked in the appropriate sidebar menu, in the same branch of the PR.
  • The contributors' names are listed in the CONTRIBUTORS file, in the same branch of the PR. Advice to have at least one contributor per page having its contact information in this CONTRIBUTORS file.
  • All relevant metadata fields in a specific page are correctly filled in (see the page metadata and the Editorial board guide). Some critical ones are listed below.
    • unique page_id (List of page IDs)
    • contributors
    • editors
    • related_pages (Related pages)
    • training
    • search_exclude must be deleted
    • description
    • supported_by (only used for Tool assembly pages)
    • coordinators(only used in national pages + they must be listed as contributors as well)
    • resources
  • Make sure that listed tools or resources are tagged in the text with the correct snippet + that its metadata is described in the tool_and_resource_list.yml file.
  • Check if relevant recipes on FAIR Cookbook can be linked to the RDMkit page that is being added/changed (Linking from RDMkit to FAIR Cookbook).
  • The content conforms to RDMkit's scope, style and templates.
  • There are no copyright issues related to the content of the page.
  • The contributors implemented the requested changes.
  • When a new page is added, a news item is added to the news.yml file, in the same branch of the PR.
  • The contributors are acknowledged for their efforts and informed about the publication of their content.
  • The PR is linked to related issues and can be merged into the main branch with no conflicts.

Please refer to the complete editor's checklist for details.

@bianchini88 bianchini88 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR includes more accurate links and a better description of the various services, and it is indeed a very welcome contribution. A couple of comments should be addressed before merging.

Comment thread _data/tool_and_resource_list.yml Resolved
Comment thread _data/tool_and_resource_list.yml Resolved
Comment thread pages/your_domain/virology.md Resolved
@Angatar
Angatar force-pushed the docs/clarify-virology-ontologies branch from 5296c1d to 2bea725 Compare September 30, 2026 13:55
@bianchini88
bianchini88 self-requested a review October 5, 2026 07:07
Comment thread _data/tool_and_resource_list.yml Resolved
@bianchini88
bianchini88 merged commit f47955a into elixir-europe:master Nov 6, 2026
4 checks passed
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.

4 participants