-
Notifications
You must be signed in to change notification settings - Fork 16
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-28748: bson-kotlin dependency #147
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.
LGTM w/ one typo fix and a suggestion
see the :ref:`Codecs <fundamentals-codecs>` documentation. You might choose | ||
Kotlin serialization if you are already familiar with the framework or | ||
you prefer to use an idiomatic Kotlin approach. | ||
To learn about how to use the ``Codec`` interface instead of the |
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.
To learn about how to use the ``Codec`` interface instead of the | |
To learn how to use the ``Codec`` interface instead of the |
and the codec registry to support Kotlin data classes, but it does | ||
not support certain POJO annotations such as ``BsonDiscriminator``, | ||
``BsonExtraElements``, and ``BsonConstructor``. To learn more, see | ||
the `bson-kotlin API documentation <{+api+}/apidocs/bson-kotlin/index.html>`__ |
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.
the `bson-kotlin API documentation <{+api+}/apidocs/bson-kotlin/index.html>`__ | |
the `bson-kotlin API documentation <{+api+}/apidocs/bson-kotlin/index.html>`__. |
* fix link * fix link * DOCSP-28748: bson-kotlin note * vale fix * JS PR fix (cherry picked from commit a5cbf35)
* fix link * fix link * DOCSP-28748: bson-kotlin note * vale fix * JS PR fix (cherry picked from commit a5cbf35)
✨ Staging URL: https://preview-mongodbmongodb.gatsbyjs.io/kotlin/master/ 🪵 Logs |
* fix link * fix link * DOCSP-28748: bson-kotlin note * vale fix * JS PR fix
* fix link * fix link * DOCSP-28748: bson-kotlin note * vale fix * JS PR fix
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-28748
Staging
Self-Review Checklist