Skip to content
This repository has been archived by the owner on Jul 27, 2024. It is now read-only.

Add SchemaJsonFormat check #512

Merged
merged 1 commit into from
Nov 29, 2021
Merged

Add SchemaJsonFormat check #512

merged 1 commit into from
Nov 29, 2021

Conversation

charlespwd
Copy link
Contributor

@charlespwd charlespwd commented Nov 25, 2021

json-formatting.mp4

Fixes #16

@charlespwd charlespwd force-pushed the feature/json-format-check branch from c125882 to c2a0d86 Compare November 25, 2021 16:11
@charlespwd charlespwd force-pushed the feature/json-format-check branch from c2a0d86 to 68780c8 Compare November 25, 2021 17:51
Copy link
Contributor

@macournoyer macournoyer left a comment

Choose a reason for hiding this comment

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

Looks good! Except for the schema? method.

@@ -166,6 +171,10 @@ def block?
block_tag? || block_body? || document?
end

def schema?
@value.is_a?(ThemeCheck::Tags::Schema)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that won't work when ThemeCheck is running in Core or SFR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why not? It's our namespace? Didn't we copy the code over?

@charlespwd charlespwd merged commit efa8eaa into main Nov 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lint/prettify JSON
2 participants