Skip to content

Conversation

@krissetto
Copy link
Contributor

- What I did
Minor test fixes that are necessary for running tests on updated ci runners and more recent versions of the engine

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

…gine version

Signed-off-by: Christopher Petito <chrisjpetito@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Feb 9, 2024

Codecov Report

Merging #4867 (30dd7c1) into master (c986d09) will not change coverage.
Report is 5 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4867   +/-   ##
=======================================
  Coverage   61.31%   61.31%           
=======================================
  Files         287      287           
  Lines       20058    20058           
=======================================
  Hits        12298    12298           
  Misses       6867     6867           
  Partials      893      893           

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Member

@krissetto should we merge this one, and then rebase the other PR?

@krissetto
Copy link
Contributor Author

@krissetto should we merge this one, and then rebase the other PR?

Sounds good to me

@krissetto krissetto marked this pull request as ready for review February 12, 2024 12:59
@krissetto krissetto merged commit d1b8893 into docker:master Feb 12, 2024
@krissetto krissetto deleted the test-fixes branch February 21, 2024 12:16
@thaJeztah thaJeztah added this to the 26.0.0 milestone Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants