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.

Sign up for GitHub

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

Jump to bottom

Skip encoding null values altogether #43

Merged

mourner merged 1 commit into master from skip-null

Jun 4, 2021

Member

mourner commented Jun 4, 2021

Closes #42. Should keep VT fully spec-compliant while avoiding the original problem of "null" strings not making sense.


          skip encoding null values altogether

fe222df

mourner requested a review from e-n-f

June 4, 2021 08:42

mourner mentioned this pull request

Don't convert null to the string "null" #39

Merged

e-n-f approved these changes

View reviewed changes

e-n-f left a comment

Thanks! I think this is the right thing to do, given the current vector tile spec and validation.

mourner merged commit d86c06f into master

mourner deleted the skip-null branch

June 4, 2021 20:25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet