Skip to content

Gregsdennis/consistency scan#1654

Merged
jdesrosiers merged 10 commits intomainfrom
gregsdennis/consistency-scan
Jan 7, 2026
Merged

Gregsdennis/consistency scan#1654
jdesrosiers merged 10 commits intomainfrom
gregsdennis/consistency-scan

Conversation

@gregsdennis
Copy link
Copy Markdown
Member

@gregsdennis gregsdennis commented Dec 21, 2025

What kind of change does this PR introduce?

edit

Issue & Discussion References

Relates to #1612

Summary

I had Copilot do a quick consistency scan of the specs. It found a few language issues, so I fixed them.

Does this PR introduce a breaking change?

No

Copy link
Copy Markdown
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

🎉

Looks like part of it was a bad merge at one point. Glad you caught all these things. I didn't check the footnote numbering changes. I'm just going to trust that it makes sense. I want to replace footnotes with something a little more friendly anyway, so it wouldn't be the end of the world if something broke there.

Comment thread specs/jsonschema-core.md Outdated
Co-authored-by: Jason Desrosiers <jdesrosi@gmail.com>
@jdesrosiers jdesrosiers merged commit 651599f into main Jan 7, 2026
5 of 6 checks passed
@jdesrosiers jdesrosiers deleted the gregsdennis/consistency-scan branch January 7, 2026 21:56
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Stable Release Development Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants