Skip to content
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

document lf-value-json's encoding of various types #2519

Merged
merged 18 commits into from
Aug 13, 2019

Conversation

S11001001
Copy link
Contributor

@S11001001 S11001001 commented Aug 13, 2019

This moves the specification of #2361 from draft into publicly-available documentation for users of the HTTP JSON API to refer to.

Discussion of formulation of the rules belongs in #2361; this PR is about documenting the rules that have been chosen.

Pull Request Checklist

NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with /AzurePipelines run to
trigger the build.

@S11001001 S11001001 marked this pull request as ready for review August 13, 2019 19:06
- suggested by @leo-da
Copy link
Contributor

@leo-da leo-da left a comment

Choose a reason for hiding this comment

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

LGTM;

@S11001001 S11001001 merged commit f77e422 into master Aug 13, 2019
@S11001001 S11001001 deleted the 2361-json-format-specification branch August 13, 2019 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants