Skip to content

Tighten JSON schema advisory checks #1120

Open
jasnow wants to merge 2 commits into
rubysec:masterfrom
jasnow:tighen-json-again
Open

Tighten JSON schema advisory checks #1120
jasnow wants to merge 2 commits into
rubysec:masterfrom
jasnow:tighen-json-again

Conversation

@jasnow

@jasnow jasnow commented Jun 15, 2026

Copy link
Copy Markdown
Member

Tighten JSON schema advisory checks

  • Disallow some fields in each JSON schema.
  • Provide enums for some fields.
  • Require 2+ lines in descriptions: field.

@jasnow

jasnow commented Jun 15, 2026

Copy link
Copy Markdown
Member Author

Fixed PR #1116 merge conflict.

Comment thread spec/schemas/gem.json Outdated
Comment thread spec/schemas/gem.json Outdated
Comment thread spec/schemas/gem.json Outdated
Comment thread spec/schemas/gem.json Outdated
@jasnow

jasnow commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

@dennispaagman - Thanks so much

@jasnow

jasnow commented Jun 18, 2026

Copy link
Copy Markdown
Member Author

A re-review would be appreciated.

@dennispaagman

Copy link
Copy Markdown
Contributor

Looks good now! I can't approve in Github itself though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants