Skip to content

Add rules of engagement for working with JSON schemas agreed by team#14

Merged
PeterBridgerKC merged 2 commits intomainfrom
rules-of-engagement
Apr 9, 2024
Merged

Add rules of engagement for working with JSON schemas agreed by team#14
PeterBridgerKC merged 2 commits intomainfrom
rules-of-engagement

Conversation

@PeterBridgerKC
Copy link
Copy Markdown
Contributor

Capturing the agreed rules of working with JSON schemas

Please do push back with suggestions the language isn't clear enough regarding use of nulls, as based on messages I've seen today I think the wording isn't clear enough here

Copy link
Copy Markdown
Contributor

@beejjacobs beejjacobs left a comment

Choose a reason for hiding this comment

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

Some suggested tweaks, generally looks clear to me.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Copy link
Copy Markdown
Contributor

@JamesP-KC JamesP-KC left a comment

Choose a reason for hiding this comment

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

All sounds very reasonable to me!

Copy link
Copy Markdown
Contributor

@DavidJWhiteDev DavidJWhiteDev left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@fabfab-kc fabfab-kc left a comment

Choose a reason for hiding this comment

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

LGTM!

@PeterBridgerKC PeterBridgerKC merged commit 351f69b into main Apr 9, 2024
@PeterBridgerKC PeterBridgerKC deleted the rules-of-engagement branch April 9, 2024 09:47
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.

5 participants