Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update the developer documentation for the e2e tests. #2381

Merged
merged 3 commits into from
Sep 25, 2023

Conversation

utam0k
Copy link
Member

@utam0k utam0k commented Sep 23, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2023

Codecov Report

Merging #2381 (ee0a344) into main (4b04abd) will not change coverage.
Report is 37 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2381   +/-   ##
=======================================
  Coverage   64.83%   64.83%           
=======================================
  Files         129      129           
  Lines       15201    15201           
=======================================
  Hits         9855     9855           
  Misses       5346     5346           

Copy link
Collaborator

@YJDoc2 YJDoc2 left a comment

Choose a reason for hiding this comment

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

overall lgtm, a minor nitpick

@YJDoc2
Copy link
Collaborator

YJDoc2 commented Sep 25, 2023

@utam0k may I ask you to do the naming changes for oci-tests -> test-oci and similar, that are done after #2330 merge?

Signed-off-by: utam0k <k0ma@utam0k.jp>
Signed-off-by: utam0k <k0ma@utam0k.jp>
Signed-off-by: utam0k <k0ma@utam0k.jp>
@utam0k utam0k merged commit 0b2f8be into youki-dev:main Sep 25, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants