Skip to content

Add gh-dash schema catalog entry - #6098

Merged
madskristensen merged 1 commit into
SchemaStore:masterfrom
Nick2bad4u:catalog/gh-dash
Jul 27, 2026
Merged

Add gh-dash schema catalog entry#6098
madskristensen merged 1 commit into
SchemaStore:masterfrom
Nick2bad4u:catalog/gh-dash

Conversation

@Nick2bad4u

@Nick2bad4u Nick2bad4u commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • register gh-dash's upstream-owned configuration schema
  • match repository-local .gh-dash.yml and .gh-dash.yaml files
  • match only the documented gh-dash/config.yml directory suffix for global/XDG configuration
  • avoid the generic config.yml matcher and the currently unused global config.yaml filename

External ownership

The upstream schema and all referenced /schema/*.json resources return JSON successfully. gh-dash has 12.1k GitHub stars as of July 23, 2026.

Dependency

The current published schema incorrectly declares defaults.dateFormat as an integer even though the released Go type and docs require a string. This draft must remain blocked until gh-dash publishes a human-authored upstream correction. Its strict no-AI policy prohibits submitting that fix from this AI-assisted workflow.

Validation

  • npm clean-install
  • npm run typecheck
  • npm run eslint
  • targeted Prettier formatting
  • SchemaStore PR audit (expected catalog-only warning only)
  • live HTTP/JSON checks for the root schema and representative relative references
  • catalog preconditions passed during node ./cli.js check; the full Windows run then reaches the existing unrelated Ajv stack overflow in cargo-lints-clippy.json

@Nick2bad4u
Nick2bad4u marked this pull request as ready for review July 26, 2026 20:33
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

@madskristensen
madskristensen merged commit bc87d45 into SchemaStore:master Jul 27, 2026
6 checks passed
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