Skip to content

Enabled strict validation in chrome-manifest.json - #5983

Merged
hyperupcall merged 8 commits into
SchemaStore:masterfrom
Vessel9817:chrome-manifest.json
Aug 31, 2026
Merged

Enabled strict validation in chrome-manifest.json#5983
hyperupcall merged 8 commits into
SchemaStore:masterfrom
Vessel9817:chrome-manifest.json

Conversation

@Vessel9817

Copy link
Copy Markdown
Contributor

Enabled strict validation

Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
See: d94f464
Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
Some developer tools may use these extraneous properties to provide type hints. In the future, **this commit should be reverted if these properties affect tests.**

This reverts commit df48eb5, replacing URLs with relative paths.

Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
@hyperupcall

Copy link
Copy Markdown
Member

Thanks, I can merge if the fileMatch is removed since it is very generic

This reverts commit 73f10c0.

Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
@Vessel9817

Copy link
Copy Markdown
Contributor Author

Thanks! Reverted in 809237e

@hyperupcall hyperupcall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the fix, just had done additional question

Comment thread src/schemas/json/chrome-manifest.json Outdated
Signed-off-by: Vessel9817 <151808241+Vessel9817@users.noreply.github.com>
@hyperupcall

Copy link
Copy Markdown
Member

Sounds good

@hyperupcall
hyperupcall merged commit 1ec8595 into SchemaStore:master Aug 31, 2026
5 checks passed
@Vessel9817
Vessel9817 deleted the chrome-manifest.json branch August 31, 2026 22:14
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