-
Notifications
You must be signed in to change notification settings - Fork 90
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
Entries schema docs #4257
Entries schema docs #4257
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.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4257 +/- ##
========================================
Coverage 37.59% 37.59%
========================================
Files 769 769
Lines 33984 33984
Branches 5156 5361 +205
========================================
Hits 12775 12775
Misses 20665 20665
Partials 544 544
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@fiskus Would you want to add your documentation about "auto-config" to this PR? |
Thank you for reminding. Let's add "auto-config" docs in a separate PR: #4258 |
@QuiltSimon @fiskus Any other comments? Okay to merge? |
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.
Changes look good! However, I think the underlying issue remains: the page is still too large and mixes tutorial content with documentation, as well as common use cases with rare cases. Perhaps we could consider splitting these into separate sections/pages in the future?
Co-authored-by: Maksim Chervonnyi <[email protected]>
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.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.
Documentation updates, to increase clarity.