Skip to content

Verlange "name" ODER "id" bei "isBasedOn"#297

Merged
acka47 merged 2 commits intomainfrom
265-isBasedOn
Jan 5, 2026
Merged

Verlange "name" ODER "id" bei "isBasedOn"#297
acka47 merged 2 commits intomainfrom
265-isBasedOn

Conversation

@acka47
Copy link
Copy Markdown
Member

@acka47 acka47 commented Dec 19, 2025

Resolves #265

@acka47 acka47 changed the title Allow "name" OR "id" for "isBasedOn" Verlange "name" ODER "id" bei "isBasedOn" Dec 19, 2025
@acka47 acka47 requested a review from MaxThomasHPI December 19, 2025 08:18
@acka47
Copy link
Copy Markdown
Member Author

acka47 commented Dec 19, 2025

Zur Erläuterung für das Review:

  • Ich habe drei neue Testdateien angelegt, um sicherzugehen, dass mit dem Schema das gewünschte Verhalten erreicht wird:
    • zwei valide: draft/examples/valid/isBasedOn_nameOnly.json & draft/examples/valid/isBasedOn_idOnly.json
    • eine invalide: draft/examples/invalid/isBasedOn_noIdOrName.json (damit die unterschiedlichenNzwecke klar werden, habe ich die bestehende invalide isBasedOn.json-Testdatei in isBasedOn_noArray.json umbenannt
  • Ich habe das Schema entsprechend aktualisiert, damit die Tests durchlaufen

Eigentlich wollte ich alles auf einmal committen, die neuen Dateien hatte ich aber aber beim ersten Commit nicht über git add ergänzt und das dann hinterher gepusht. Ich werde dann die beiden Commits ja beim Mergen squashen.

@acka47 acka47 merged commit d94c6fe into main Jan 5, 2026
2 checks passed
@acka47 acka47 deleted the 265-isBasedOn branch January 5, 2026 09:48
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.

Verlange name ODER id bei isBasedOn

2 participants