-
Notifications
You must be signed in to change notification settings - Fork 412
Add support for checked
#151
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
Conversation
72a149f
to
b184e1c
Compare
43ce7a1
to
e1367f4
Compare
@natebosch @jakemac53 – I split this out into commits that should be easy to consume individually |
What is |
/// for large object graphs. | ||
/// [Map] and [List] instances created during serialization. | ||
/// | ||
/// This will increase the code size, but it may improve runtime performance, |
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.
These line lengths look inconsistent. Would [Map]
have fit within 80 characters on line 45 or is this line going over 80 characters?
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.
I often wrap at the sentence start. It doesn't affect markdown rendering, but it doe make it easier to edit sentences without rewrapping the entire paragraph.
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.
neither of these lines are wrapped at a sentence start...
No description provided.