Skip to content

Add examples and media/relation types for Argo, openEO, Jupyter and CWL - #10

Open
m-mohr wants to merge 3 commits into
mainfrom
application-workflow-examples
Open

Add examples and media/relation types for Argo, openEO, Jupyter and CWL#10
m-mohr wants to merge 3 commits into
mainfrom
application-workflow-examples

Conversation

@m-mohr

@m-mohr m-mohr commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

@m-mohr
m-mohr marked this pull request as draft August 10, 2026 17:47
@m-mohr
m-mohr force-pushed the application-workflow-examples branch from 03e0699 to e0bff4b Compare August 10, 2026 21:03
@m-mohr
m-mohr force-pushed the application-workflow-examples branch from e0bff4b to 5432137 Compare August 11, 2026 14:12
@m-mohr
m-mohr marked this pull request as ready for review September 8, 2026 15:46
@m-mohr
m-mohr requested a lite review from Copilot September 8, 2026 15:49

This comment was marked as resolved.

@m-mohr
m-mohr force-pushed the application-workflow-examples branch from 95a5164 to 95996f5 Compare September 8, 2026 16:04
@m-mohr
m-mohr requested a lite review from Copilot September 8, 2026 16:04

This comment was marked as resolved.

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.

🔵 Needs a closer look

At least one newly added example contains a broken raw GitHub URL format, and there are a couple of small but concrete documentation wording issues to fix.

Review details

Suppressed comments (3)

Previously missed (2) — in code that hasn't changed since the last review.

examples/stac/cwl-application-package.json:47

  • The raw.githubusercontent.com URL format is incorrect here: raw URLs don’t include /refs/heads/…, so this example link is likely broken.
    README.md:21
  • The NOTE block has a small grammatical issue: “not best suited” should be “not well suited”.

This issue also appears on line 66 of the same file.

README.md:66

  • This sentence reads awkwardly in singular form (“any new container”); it should be plural to match “any new …”.
Please submit any new container via GitHub issues so that we can make a list of names everyone can reuse.
  • Files reviewed: 10/10 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

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.

2 participants