-
-
Notifications
You must be signed in to change notification settings - Fork 247
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix some inconsistent indentation #635
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Welcome to the JSON Schema Community. Thanks a lot for creating your first pull request!! 🎉🎉 We are so excited you are here! We hope this is only the first of many! For more details check out README.md file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for finding and fixing this @theory !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes you made look fine. Just a couple more, and I think it's good.
"state": { "type": "string" } | ||
"street_address": { "type": "string" }, | ||
"city": { "type": "string" }, | ||
"state": { "type": "string" } | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be pulled back by two spaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 1ae08de.
@@ -263,10 +262,9 @@ the subschema on the highlighted part of the following schema. | |||
"$id": "https://example.com/schemas/address", | |||
"type": "object", | |||
"properties": { | |||
"street_address": | |||
{ | |||
"street_address": { | |||
"$anchor": "street_address", "type": "string" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Each property should have its own line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 1ae08de.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!!
What kind of change does this PR introduce?
Bug fix (indentation correction)
Issue Number:
N/A
Screenshots/videos:
N/A
If relevant, did you update the documentation?
https://json-schema.org/understanding-json-schema/structuring
Summary
I was reading the structuring doc and found the indentation in a couple of the code examples confusing. I've adjusted it to be more like other examples of the address schema in the doc, making easier to read.
Does this PR introduce a breaking change?
I hope not!