Skip to content

Add Nextflow training collection - #31

Closed
GiorgiaMori wants to merge 21 commits into
AustralianBioCommons:mainfrom
GiorgiaMori:main
Closed

GiorgiaMori wants to merge 21 commits into
AustralianBioCommons:mainfrom
GiorgiaMori:main

Conversation

@GiorgiaMori

Copy link
Copy Markdown
Collaborator

Adding the Australian BioCommons Nextflow training collection

Copilot AI lite review requested due to automatic review settings September 14, 2026 04:44
@GiorgiaMori GiorgiaMori added the enhancement New feature or request label Sep 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Existing resources and the /nextflow route may be lost, with additional metadata and accessibility issues remaining.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds an Australian BioCommons Nextflow training collection with curated learning resources.

Changes:

  • Adds a card-based training collection.
  • Updates sidebar navigation.
  • Replaces the previous Nextflow page and resource entries.
File summaries
File Summary
pages/nextflow.md Removes the previous collection page.
pages/nextflow_training.md Adds the new training collection layout.
_data/sidebars/main.yml Updates navigation to the new collection.
_data/all_content_list.yml Adds training metadata and replaces existing Nextflow entries.
Review details

Suppressed comments (3)

_data/sidebars/main.yml:15

  • Replacing the existing /nextflow sidebar target with /nextflow_training removes the public /nextflow route because pages use /:basename permalinks. Existing bookmarks or external links to the current collection will therefore return 404; please retain a compatibility redirect/page for /nextflow while exposing the renamed collection.
        url: /nextflow_training

pages/nextflow_training.md:59

  • The preceding Markdown heading already receives the training-materials slug in Jekyll/Kramdown, so this wrapper creates a duplicate HTML id. Duplicate IDs make the tile anchor target ambiguous; remove the wrapper id and let the heading provide the anchor.
<div class="row row-cols-1 row-cols-md-2 g-4 mb-5" id="training-materials">

pages/nextflow_training.md:75

  • Every card exposes the same link name, Open resource; screen-reader users navigating a links list cannot distinguish the destinations. Add the resource name to the link's accessible name while retaining the visible label.
						<a href="{{ resource.url }}">Open resource</a>
  • Files reviewed: 4/4 changed files
  • Comments generated: 5
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread _data/all_content_list.yml
Comment thread _data/all_content_list.yml
Comment thread _data/all_content_list.yml Outdated
Comment thread pages/nextflow_training.md Outdated
Comment thread pages/nextflow_training.md Outdated
Comment thread pages/nextflow_training.md Outdated
Comment thread pages/nextflow_training.md
Comment thread pages/nextflow_training.md
Comment thread pages/nextflow_training.md
Comment thread pages/nextflow_training.md
Comment thread pages/nextflow_training.md Outdated
Comment thread _data/all_content_list.yml Outdated
Comment thread _data/all_content_list.yml Outdated
Comment thread _data/all_content_list.yml Outdated
Comment thread _data/all_content_list.yml Outdated
Comment thread _data/all_content_list.yml Outdated
Comment thread _data/all_content_list.yml Outdated
provider: Sydney Informatics Hub and Australian BioCommons
topics: self-paced learning
type: static website
url: https://zenodo.org/records/17694728

@burkemlou burkemlou Sep 24, 2026 •

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.

To check with Rahul (not necessarily right now but certainly sooner rather than later):

Is content in tiles accessible to search engines and AI?

Is content in tiles accessible to screen readers?

GiorgiaMori and others added 9 commits September 28, 2026 17:56
Co-authored-by: Melissa Burke <68895620+burkemlou@users.noreply.github.com>
Co-authored-by: Melissa Burke <68895620+burkemlou@users.noreply.github.com>
Updated headings and removed navigation tiles for better structure and clarity.
Updated resource links to include aria-label for accessibility.
Updated headings and added navigation tiles for self-paced learning and training materials.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants