Skip to content
This repository was archived by the owner on Jun 28, 2024. It is now read-only.

Make it clear that new tests should go to the kata-containers repo#5718

Open
fidencio wants to merge 2 commits into
kata-containers:mainfrom
fidencio:topic/make-it-clear-that-new-tests-should-go-to-kata-containers
Open

Make it clear that new tests should go to the kata-containers repo#5718
fidencio wants to merge 2 commits into
kata-containers:mainfrom
fidencio:topic/make-it-clear-that-new-tests-should-go-to-kata-containers

Conversation

@fidencio

Copy link
Copy Markdown
Member

Let's make it as explicit as possible that new tests should be developed
for the kata-containers repo instead of this one.

Fixes: #5717

@fidencio fidencio requested a review from a team as a code owner July 18, 2023 14:55
@fidencio

Copy link
Copy Markdown
Member Author

/test

@katacontainersbot katacontainersbot added the size/small Small and simple task label Jul 18, 2023
Comment thread README.md
fidencio added 2 commits July 18, 2023 18:00
It's been deprecated for some time already.

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
Let's make it as explicit as possible that new tests should be developed
for the `kata-containers` repo instead of this one.

Fixes: kata-containers#5717

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
@fidencio fidencio force-pushed the topic/make-it-clear-that-new-tests-should-go-to-kata-containers branch from 48de655 to 39eebfb Compare July 18, 2023 16:00
@fidencio

Copy link
Copy Markdown
Member Author

/test

@stevenhorsman stevenhorsman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks for the go get fix whilst you were in the README

@studychao studychao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, a simple comment.
And also, do we plan to migrate all tests into Kata main repo (including tests like spelling check, license check)?

Comment thread README.md

**NOTE:** If you're working on adding new tests to Kata Containers, **please**,
keep in mind that we're in the process of migrating the tests from this repo to
the kata-containers one.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe a link here would be better?
[Kata Containers](https://github.com/kata-containers)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size/small Small and simple task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

README: Add note about moving the tests to the kata-containers repo

4 participants