Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(deps): Bump github.com/BurntSushi/toml from 0.4.1 to 1.2.0 #11605

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 2, 2022

Bumps github.com/BurntSushi/toml from 0.4.1 to 1.2.0.

Release notes

Sourced from github.com/BurntSushi/toml's releases.

Release v1.2.0

This release adds a few additional features:

  • Support encoding and decoding json.Number, making interoperability with JSON when using json.Encoder.UseNumber() easier.

  • Support time.Duration string format; an integer will be interpreted as nanoseconds (like before), but a string like "5h" will be parsed. On encoding the string format is used.

  • The omitempty struct tag now also works for structs with all zero values, for example an empty time.Time. A struct is considered "empty" when all fields (exported and private) are the zero value.

  • Allow using interface{} (or any) as the map key when decoding.

And some fixes:

  • Fix encoding.TextUnmarshaler not working for map values.

  • Make encoding.TextUnmarshaler and toml.Unmarshaler work if MarshalText() or MarshalTOML() have a pointer receiver.

  • Error out on nil returns from MarshalTOML/MarshalText; before they would get encoded as invalid TOML (keyname =).

  • Fix a panic where it would try to encode array = [{}, 0] as:

    [[array]]
    [[array]]
    

    Which isn't valid as 0 isn't a table.

  • Some better error reporting for some errors.

v1.1.0

Just a few bugfixes:

  • Skip fields with toml:"-" even when they're unsupported types. Previously something like this would fail to encode due to func being an unsupported type:

    struct {
        Str  string `toml:"str"
        Func func() `toml:"-"`
    }
    
  • Multiline strings can't end with \. This is valid:

    # Valid
    key = """ foo \
    """
    

    Invalid

    key = """ foo \ """

  • Don't quote values in TOMLMarshaler. Previously they would always include quoting (e.g. "value"), while the entire point of this interface is to bypass that.

v1.0.0

... (truncated)

Commits
  • 2e74712 Simplify test case
  • e2f6fa2 Allow using interface{} as map keys when decoding
  • f0ccf71 Merge pull request #358 from kkHAIKE/encode_MarshalText_fix
  • c03a31c Small stylistic updates
  • 6805a3d add test for embed field
  • 0ae83fe replace some Interface() check when encode private embed struct
  • eaf0d98 add test
  • d138d7a change eindirect behave match with indirect from decode
  • 0a9f2b0 Don't panic encoding arrays where the first item is a table, but others aren't
  • 28ff18d Merge pull request #356 from BurntSushi/empty
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [github.com/BurntSushi/toml](https://github.com/BurntSushi/toml) from 0.4.1 to 1.2.0.
- [Release notes](https://github.com/BurntSushi/toml/releases)
- [Commits](BurntSushi/toml@v0.4.1...v1.2.0)

---
updated-dependencies:
- dependency-name: github.com/BurntSushi/toml
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Aug 2, 2022
@telegraf-tiger telegraf-tiger bot added the chore label Aug 2, 2022
@telegraf-tiger
Copy link
Contributor

telegraf-tiger bot commented Aug 2, 2022

@srebhan srebhan self-assigned this Aug 3, 2022
@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Aug 3, 2022
Copy link
Contributor

@powersj powersj left a comment

Choose a reason for hiding this comment

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

It slightly scares me to make this large of a jump, given our toml parser is not in line with that level, but given this is only used in the shim + execd, I think this is lower risk.

@powersj powersj merged commit 756001b into master Aug 3, 2022
@powersj powersj deleted the dependabot/go_modules/github.com/BurntSushi/toml-1.2.0 branch August 3, 2022 18:06
reimda pushed a commit that referenced this pull request Aug 15, 2022
reimda pushed a commit that referenced this pull request Aug 16, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
chore dependencies Pull requests that update a dependency file ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants