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

Update module github.com/pelletier/go-toml/v2 to v2.1.0 #1456

Merged
merged 1 commit into from
Sep 12, 2023
Merged

Update module github.com/pelletier/go-toml/v2 to v2.1.0 #1456

merged 1 commit into from
Sep 12, 2023

Conversation

tpapagian
Copy link
Member

Fixes issues in #1159

@tpapagian
Update module github.com/pelletier/go-toml/v2 to v2.1.0
712bb91 
Fixes issues in #1159

Signed-off-by: Anastasios Papagiannis <[email protected]>
@tpapagian tpapagian added the release-note/misc This PR makes changes that have no direct user impact. label Sep 12, 2023
kevsecurity
kevsecurity approved these changes Sep 12, 2023
View reviewed changes
Copy link
Contributor

@kevsecurity kevsecurity left a comment

Choose a reason for hiding this comment

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

LGTM. Close 1159.

@tpapagian tpapagian marked this pull request as ready for review September 12, 2023 11:31
@tpapagian tpapagian requested a review from a team as a code owner September 12, 2023 11:31
mtardy
mtardy approved these changes Sep 12, 2023
View reviewed changes
Copy link
Member

@mtardy mtardy left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks a lot for taking the time to do this. No need to close the other issue, it will autoclose on next renovate run :) (20h for us, 11am for pacific time or we can trigger manually)

@mtardy mtardy merged commit 8d27aec into main Sep 12, 2023
27 of 28 checks passed
@mtardy mtardy deleted the pr/apapag/update_go-toml branch September 12, 2023 12:32
@p-linnane p-linnane mentioned this pull request Nov 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kevsecurity kevsecurity kevsecurity approved these changes

@mtardy mtardy mtardy approved these changes

@tixxdz tixxdz Awaiting requested review from tixxdz tixxdz is a code owner automatically assigned from cilium/tetragon

Assignees
No one assigned
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tpapagian @mtardy @kevsecurity