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

Migrate to YAML encoder gopkg.in/yaml.v3 #69

Closed
arcticicestudio opened this issue Jul 13, 2019 · 1 comment
Closed

Migrate to YAML encoder gopkg.in/yaml.v3 #69

arcticicestudio opened this issue Jul 13, 2019 · 1 comment
Labels
context-config scope-compatibility scope-maintainability scope-performance scope-security scope-stability type-improvement
Milestone
0.4.0

Comments

@arcticicestudio
Copy link
Owner

arcticicestudio commented Jul 13, 2019 •
edited
Loading

Epic: #33

Currently the github.com/ghodss/yaml package is used as encoder for YAML files. It was evaluated and chosen over the gopkg.in/yaml.v2 package because it fixed some problems and misleading YAML specification implementations while also simplifying the usage of both JSON and YAML at the same time.
Unfortunately the package is not actively maintained anymore and also has some problems and usage edge cases while the gopkg.in/yaml package is actively maintained and improved the YAML specification compatibility a lot.
Therefore snowsaw will migrate to the latest major version gopkg.in/yaml.v3 replacing the currently used package.
@arcticicestudio arcticicestudio added this to the 0.4.0 milestone Jul 13, 2019
@arcticicestudio arcticicestudio self-assigned this Jul 13, 2019
arcticicestudio added a commit that referenced this issue Jul 13, 2019
@arcticicestudio
Migrate to YAML encoder `gopkg.in/yaml.v3
dea6ab5 
Previously the `github.com/ghodss/yaml` (1) package was used as encoder
for YAML (2) files. It was evaluated and choosen over the
`gopkg.in/yaml.v2` package because it fixed some problems and misleading
YAML specification implementations while also simplifying the usage of
both JSON (3) and YAML (2) at the same time.
Unfortunately the package is not actively maintained anymore and also
has some problems and usage edge cases while the `gopkg.in/yaml` package
is actively maintained and improved the YAML specification compatibility
a lot.
Therefore snowsaw migrated to the latest major version
`gopkg.in/yaml.v3` (4) replacing the previously used package.

References:
  (1) https://github.com/ghodss/yaml
  (2) https://yaml.org
  (3) https://www.json.org
  (4) https://github.com/go-yaml/yaml/tree/v3

Epic GH-33
Resolves GH-69
@arcticicestudio
Copy link
Owner Author

Resolved in dea6ab5

@arcticicestudio arcticicestudio removed their assignment Jul 13, 2019
@arcticicestudio arcticicestudio mentioned this issue Jul 14, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
context-config scope-compatibility scope-maintainability scope-performance scope-security scope-stability type-improvement
Projects
None yet
Milestone
0.4.0
Development

No branches or pull requests
1 participant
@arcticicestudio