Skip to content

Use config schema published on dodona.be#54

Merged
chvp merged 1 commit intomainfrom
fix/use-dodona-published-schema
Mar 2, 2026
Merged

Use config schema published on dodona.be#54
chvp merged 1 commit intomainfrom
fix/use-dodona-published-schema

Conversation

@chvp
Copy link
Copy Markdown
Member

@chvp chvp commented Feb 27, 2026

The schema here actually contains errors (no intermediate names property under description. Since we will be hosting a schema on dodona.be anyway, just refer to that.

Depends on https://github.com/dodona-edu/dodona/pull/7693

The schema here actually contains errors (no intermediate names property under description. Since we will be hosting a schema on dodona.be anyway, just refer to that.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the VS Code extension’s JSON schema validation to rely on the canonical config schema hosted on dodona.be, removing the bundled local schema that had known errors.

Changes:

  • Removed the outdated/broken local config.schema.json.
  • Updated package.json JSON validation to use https://dodona.be/schemas/config.json.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
schemas/config.schema.json Removes the previously bundled schema that contained inaccuracies.
package.json Points VS Code JSON validation for config.json / dirconfig.json to the hosted schema on dodona.be.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@chvp chvp merged commit a9ce816 into main Mar 2, 2026
5 checks passed
@chvp chvp deleted the fix/use-dodona-published-schema branch March 2, 2026 14:40
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.

3 participants