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

switch to gopkg.in/yaml.v3 #4178

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

switch to gopkg.in/yaml.v3 #4178

wants to merge 3 commits into from
+21 −10,064

Commits on Dec 14, 2023

  1. switch to gopkg.in/yaml.v3 

    Looks like most other projects switched over to v3, so let's follow suit.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
    @thaJeztah @milosgajdos
    thaJeztah authored and milosgajdos committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    51002c6 View commit details
    Browse the repository at this point in the history

  2. fix: update the type switch when reading config variables 

    gopkg.in/yaml.v3 changed the way it parses YAML.
See the following commit for epxplanation:
go-yaml/yaml@14f799f

Signed-off-by: Milos Gajdos <milosthegajdos@gmail.com>
    @milosgajdos
    milosgajdos committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    3f5546b View commit details
    Browse the repository at this point in the history

  3. fix: update code that switch types on invalid params 

    Signed-off-by: Milos Gajdos <milosthegajdos@gmail.com>
    @milosgajdos
    milosgajdos committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    d467325 View commit details
    Browse the repository at this point in the history