Skip to content

some minor cleanups#3978

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:minor_cleanups
Apr 9, 2023
Merged

some minor cleanups#3978
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:minor_cleanups

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Had these in a branch I'm working on, but they were not directly related, so opening a PR to get these changes out of the way.

@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Jan 18, 2023
@thaJeztah thaJeztah added this to the 23.0.0 milestone Jan 18, 2023
@thaJeztah thaJeztah requested a review from vvoland January 18, 2023 16:42
@thaJeztah
Copy link
Copy Markdown
Member Author

Hmm.. CI doesn't like it 😂

Possibly screwed up something somewhere; I'll check when I'm back at my computer

@thaJeztah thaJeztah modified the milestones: 23.0.0, 23.0.1 Feb 2, 2023
@thaJeztah thaJeztah modified the milestones: 23.0.1, v-next Feb 9, 2023
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 9, 2023

Codecov Report

Merging #3978 (8468eef) into master (d49b8ff) will not change coverage.
The diff coverage is 80.00%.

❗ Current head 8468eef differs from pull request most recent head f36e4b0. Consider uploading reports for the commit f36e4b0 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3978   +/-   ##
=======================================
  Coverage   59.03%   59.03%           
=======================================
  Files         288      288           
  Lines       24776    24776           
=======================================
  Hits        14627    14627           
  Misses       9265     9265           
  Partials      884      884           

- containerConfig collided with the containerConfig type
- warning collided with the warning const

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Copy Markdown
Member Author

dropped the first commit for now; all green; bringing this in.

@thaJeztah thaJeztah merged commit 5acb649 into docker:master Apr 9, 2023
@thaJeztah thaJeztah deleted the minor_cleanups branch April 9, 2023 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants