Skip to content

chore: replace github.com/ghodss/yaml with sigs.k8s.io/yaml#175

Merged
zachaller merged 2 commits intoargoproj:masterfrom
Juneezee:chore/yaml
May 5, 2023
Merged

chore: replace github.com/ghodss/yaml with sigs.k8s.io/yaml#175
zachaller merged 2 commits intoargoproj:masterfrom
Juneezee:chore/yaml

Conversation

@Juneezee
Copy link
Contributor

Looks great! It would also be good to update the [notification-engine (https://github.com/argoproj/notifications-engine) as well.

Originally posted by @blakepettersson in argoproj/argo-cd#13292 (comment)

This PR is part of argoproj/argo-cd#13292 to completely replace github.com/ghodss.yaml with sigs.k8s.io/yaml.

At the time of making this commit, the package `github.com/ghodss/yaml`
is no longer actively maintained.

`sigs.k8s.io/yaml` is a permanent fork of `ghodss/yaml` and is actively
maintained by Kubernetes SIG.

Reference: argoproj/argo-cd#13292 (comment)
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
@codecov
Copy link

codecov bot commented Apr 20, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (c028d2f) 52.90% compared to head (3241da3) 52.90%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   52.90%   52.90%           
=======================================
  Files          33       33           
  Lines        2807     2807           
=======================================
  Hits         1485     1485           
  Misses       1101     1101           
  Partials      221      221           
Impacted Files Coverage Δ
pkg/api/config.go 50.96% <ø> (ø)
pkg/cmd/context.go 44.44% <ø> (ø)
pkg/services/googlechat.go 59.22% <ø> (ø)
pkg/services/services.go 25.51% <ø> (ø)
pkg/subscriptions/annotations.go 77.14% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@blakepettersson blakepettersson left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Juneezee
Copy link
Contributor Author

Juneezee commented May 5, 2023

Hi @zachaller, would you mind taking a few minutes to review this PR too? Thanks 😃

@zachaller zachaller merged commit 8f2ae5e into argoproj:master May 5, 2023
mayzhang2000 pushed a commit that referenced this pull request May 9, 2023
At the time of making this commit, the package `github.com/ghodss/yaml`
is no longer actively maintained.

`sigs.k8s.io/yaml` is a permanent fork of `ghodss/yaml` and is actively
maintained by Kubernetes SIG.

Reference: argoproj/argo-cd#13292 (comment)

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
zachaller added a commit that referenced this pull request Jul 12, 2023
* self-service from event

* chore: replace github.com/ghodss/yaml with sigs.k8s.io/yaml (#175)

At the time of making this commit, the package `github.com/ghodss/yaml`
is no longer actively maintained.

`sigs.k8s.io/yaml` is a permanent fork of `ghodss/yaml` and is actively
maintained by Kubernetes SIG.

Reference: argoproj/argo-cd#13292 (comment)

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* feat: Adding new PagerDuty integration based on Events API v2 (#105)

Signed-off-by: Eric Tendian <erictendian@gmail.com>

* self-service from event

* self-service from event

* self-service from event

* self-service from event

* self-service from event

* clean up factory.go

* error handleing in factory.go

* error handleing in factory.go

* error handleing in factory.go

* error handleing in factory.go

* error handleing in factory.go

* error handleing in factory.go

* error handleing in factory.go

* better caching

* better caching

* refactor

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* unused

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* lint

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* bump ci

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* small fixes

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* rename

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* rename

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* rename

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* change log line

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* update tests

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* continue on error

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* continue on errors

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* add godoc

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* continue on error

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* fix error by going back to non namespace support

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* fix log

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

* improve log on error

Signed-off-by: zachaller <zachaller@users.noreply.github.com>

---------

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
Signed-off-by: Eric Tendian <erictendian@gmail.com>
Signed-off-by: zachaller <zachaller@users.noreply.github.com>
Co-authored-by: May Zhang <may_zhang@intuit.com>
Co-authored-by: Eng Zer Jun <engzerjun@gmail.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Co-authored-by: Eric Tendian <erictendian@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants