Add Nextflow training collection - #31
Closed
GiorgiaMori wants to merge 21 commits into
Closed
GiorgiaMori wants to merge 21 commits into
GiorgiaMori wants to merge 21 commits into
Conversation
There was a problem hiding this comment.
🟡 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
/nextflowsidebar target with/nextflow_trainingremoves the public/nextflowroute because pages use/:basenamepermalinks. Existing bookmarks or external links to the current collection will therefore return 404; please retain a compatibility redirect/page for/nextflowwhile exposing the renamed collection.
url: /nextflow_training
pages/nextflow_training.md:59
- The preceding Markdown heading already receives the
training-materialsslug in Jekyll/Kramdown, so this wrapper creates a duplicate HTMLid. Duplicate IDs make the tile anchor target ambiguous; remove the wrapperidand 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.
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
burkemlou
reviewed
Sep 24, 2026
| provider: Sydney Informatics Hub and Australian BioCommons | ||
| topics: self-paced learning | ||
| type: static website | ||
| url: https://zenodo.org/records/17694728 |
Collaborator
There was a problem hiding this comment.
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?
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.
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.
Adding the Australian BioCommons Nextflow training collection