Skip to content
This repository was archived by the owner on Nov 2, 2023. It is now read-only.

Updated primer #487

Merged
merged 2 commits into from
Nov 14, 2022
Merged

Updated primer #487

merged 2 commits into from
Nov 14, 2022

Conversation

Relequestual
Copy link
Member

Opened in favour of #423

Opened in favour of #423
@netlify
Copy link

netlify bot commented Nov 3, 2022

Deploy Preview for condescending-hopper-c3ed30 ready!

Name Link
🔨 Latest commit 645fd3c
🔍 Latest deploy log https://app.netlify.com/sites/condescending-hopper-c3ed30/deploys/63721dc324a26c000760411e
😎 Deploy Preview https://deploy-preview-487--condescending-hopper-c3ed30.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

index.md Outdated
@@ -5,7 +5,7 @@ permalink: /
---


**JSON Schema** is a vocabulary that allows you to **annotate** and **validate** JSON documents.
**JSON Schema** is a declarative ___domain specific language that allows you to **annotate** and **validate** JSON documents. JSON Schema enables the confident and reliable use of the JSON data format.
Copy link
Contributor

Choose a reason for hiding this comment

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

From the discussion on the old PR, it looks like @awwright and @jdesrosiers agreed that this should be change to just "declarative language", which I also prefer.

Copy link
Member

Choose a reason for hiding this comment

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

Either version works for me, but I think leaving out "___domain specific" is more accessible and it effectively says the same thing either way.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah. Even though I know what "___domain specific" means it still threw me a bit trying to figure out exactly how it was meant here.

@Relequestual Relequestual merged commit 0d3e024 into main Nov 14, 2022
@Relequestual Relequestual deleted the update-primer branch November 14, 2022 11:32
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.

3 participants