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.

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

bump sirupsen/logrus to 1.9.0 #6316

Closed
wants to merge 1 commit into from
Closed

bump sirupsen/logrus to 1.9.0 #6316

wants to merge 1 commit into from

Conversation

yanggangtony
Copy link
Contributor

As src/runtime/go.mod update the sirupsen/logrus to 1.9.0.
Also current latest version is 1.9.0.

So update tools/log-parser/go.mod to match that.

@katacontainersbot
Copy link
Contributor

Can one of the admins verify this patch?

@yanggangtony
Copy link
Contributor Author

Hello , can anybody review these pr?
Its have been a long time.

@c3d
Copy link
Member

c3d commented Mar 27, 2023

Hi @yanggangtony, please follow the guidelines outlined here: https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format

Notably, you need to have an associated issue, and your patch needs to mention it in a Fixes: #<issue number> line.

@c3d
Copy link
Member

c3d commented Mar 27, 2023

/test

@yanggangtony yanggangtony requested a review from a team as a code owner March 27, 2023 11:46
@katacontainersbot katacontainersbot added the size/small Small and simple task label Mar 27, 2023
@yanggangtony
Copy link
Contributor Author

Hi @yanggangtony, please follow the guidelines outlined here: https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format

Notably, you need to have an associated issue, and your patch needs to mention it in a Fixes: #<issue number> line.

Ah, thanks for reminder.
Rebased it.

Fixes: #6530

Signed-off-by: yanggang <[email protected]>
@yanggangtony yanggangtony deleted the logrus-upgread branch June 13, 2023 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants