Skip to content

fix wrong notes for func GetSandboxesStoragePathRust()#6355

Merged
cmaf merged 1 commit intokata-containers:mainfrom
yanggangtony:fix-wrong-notes
Mar 3, 2023
Merged

fix wrong notes for func GetSandboxesStoragePathRust()#6355
cmaf merged 1 commit intokata-containers:mainfrom
yanggangtony:fix-wrong-notes

Conversation

@yanggangtony
Copy link
Contributor

fix wrong notes for func GetSandboxesStoragePathRust()

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Feb 23, 2023
@katacontainersbot
Copy link
Contributor

Can one of the admins verify this patch?

Copy link
Member

@Tim-0731-Hzt Tim-0731-Hzt left a comment

Choose a reason for hiding this comment

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

Thanks @yanggangtony LGTM

@liubin
Copy link
Member

liubin commented Feb 27, 2023

Thank you @yanggangtony , before this PR being merged, you need fix your commit message https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format

@yanggangtony
Copy link
Contributor Author

@liubin
thanks for review.

@Tim-0731-Hzt
Copy link
Member

@yanggangtony, you need to raise an issue for this PR, and add Fixes:#XXX in the commit

Fix wrong notes for func GetSandboxesStoragePathRust()

Fixes: #6394

Signed-off-by: yanggang <[email protected]>
@yanggangtony
Copy link
Contributor Author

@Tim-0731-Hzt
ah, thanks for the advice.
update it.

@cmaf cmaf linked an issue Mar 2, 2023 that may be closed by this pull request
@cmaf
Copy link

cmaf commented Mar 2, 2023

/test

@c3d
Copy link
Member

c3d commented Mar 3, 2023

@cmaf It looks like the jenkins-ci-s390x-unit-ubuntu-20-04 job completed several hours ago, but somehow github did not see it, so I cannot merge this. What is the procedure in that case?

@BbolroC
Copy link
Member

BbolroC commented Mar 3, 2023

/test-s390x-unit

@cmaf cmaf merged commit ebe916b into kata-containers:main Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/tiny Smallest and simplest task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

modify the code notes.

7 participants