Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

remove dependency on github.com/ghodss/yaml#1385

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
lahabana wants to merge3 commits intogruntwork-io:main
base:main
Choose a base branch
Loading
fromlahabana:remove-ghodss/yaml

Conversation

@lahabana
Copy link
Contributor

Description

Library is not maintained anymore and the same utility is provided in k8s which the project already depends on

TODOs

Read theGruntwork contribution guidelines.

  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Ensure any 3rd party code adheres with ourlicense policy or delete this line if its not applicable.
  • Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Remove dependency on github.com/ghodss/yaml which is not maintained anymore

Migration Guide

N/A

so0k reacted with heart emoji
@denis256
Copy link
Member

pre-commit installed at .git/hooks/pre-commit[INFO] Initializing environment for https://github.com/gruntwork-io/pre-commit.goimports................................................................Failed- hook id: goimports- files were modified by this hookmodules/helm/template.go

@lahabana
Copy link
ContributorAuthor

@denis256 updated sorry it took so long

Library is not supported anymore and the same utilityis provided in k8s which the project already depends onSigned-off-by: Charly Molter <charly.molter@konghq.com>
@lahabana
Copy link
ContributorAuthor

@denis256 updated as well

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

Reviewers

@denis256denis256Awaiting requested review from denis256denis256 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lahabana@denis256

[8]ページ先頭

©2009-2025 Movatter.jp