-
Notifications
You must be signed in to change notification settings - Fork 6
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
DOCSP-42813: codecs #43
Conversation
👷 Deploy request for docs-kotlin-sync pending review.Visit the deploys page to approve it
|
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.
Nice work ✨ left a few small comments but LGTM!
source/data-formats/codecs.txt
Outdated
* - Parameter Type | ||
- Description | ||
|
||
* - ``bsonReader`` |
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.
[question] Should this be reader
similar to the writer
parameter in the previous table? I noticed reader
is in the example and in the API doc but not specifically described in this table.
source/data-formats/codecs.txt
Outdated
enum values to BSON booleans. | ||
|
||
- ``colorTemperature``: Describes the color of the device light in | ||
Kelvins as an ``Int`` value. For this field, use the ``IntegerCodec`` |
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.
[nit] I think this is usually singular if capitalized, or lowercase when plural:
Kelvins as an ``Int`` value. For this field, use the ``IntegerCodec`` | |
kelvins as an ``Int`` value. For this field, use the ``IntegerCodec`` |
or
Kelvins as an ``Int`` value. For this field, use the ``IntegerCodec`` | |
Kelvin as an ``Int`` value. For this field, use the ``IntegerCodec`` |
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.
LGTM
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-42813
Staging - https://preview-mongodbrustagir.gatsbyjs.io/kotlin-sync/DOCSP-42813-codecs/data-formats/codecs/
Self-Review Checklist