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.

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

Support multi-field encoding using zap.Inline #912

Merged
merged 4 commits into from Mar 23, 2021
Merged

Support multi-field encoding using zap.Inline #912

merged 4 commits into from Mar 23, 2021

Commits on Feb 24, 2021

  1. Support multi-field encoding using zap.InlineObject 

    Fixes #876

Currently, a `zap.Field` can only represent a single key-value. Add
`zap.InlineObject` so to allow adding multiple fields to the current
namespace from a type implementing `zap.ObjectMarshaler`.

This also solves a more general problem: a single `zap.Field` can now
be used to add multiple key/value pairs.
    @prashantv
    prashantv committed Feb 24, 2021
    Copy the full SHA
    c6dbd12 View commit details
    Browse the repository at this point in the history

Commits on Mar 2, 2021

  1. Add nested object test

    @prashantv
    prashantv committed Mar 2, 2021
    Copy the full SHA
    fff4bfa View commit details
    Browse the repository at this point in the history

  2. s/InlineObject/Inline

    @prashantv
    prashantv committed Mar 2, 2021
    Copy the full SHA
    cdfbaec View commit details
    Browse the repository at this point in the history

Commits on Mar 19, 2021

  1. Update Inline comment

    @prashantv
    prashantv committed Mar 19, 2021
    Copy the full SHA
    f17ba22 View commit details
    Browse the repository at this point in the history