Skip to content

Tighten JSON schema advisory checks - #1120

Merged
jasnow merged 2 commits into
rubysec:masterfrom
jasnow:tighen-json-again
Jun 19, 2026
Merged

Tighten JSON schema advisory checks #1120
jasnow merged 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.

@jasnow
jasnow merged commit f4a608c into rubysec:master Jun 19, 2026
2 checks passed
@jasnow
jasnow deleted the tighen-json-again branch June 19, 2026 11:22
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