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

Folder refactor and new CI compose test #26

Merged
merged 2 commits into from
Jan 23, 2023
Merged

Folder refactor and new CI compose test #26

merged 2 commits into from
Jan 23, 2023

Conversation

glimchb
Copy link
Member

@glimchb glimchb commented Jan 23, 2023

@glimchb glimchb requested a review from a team as a code owner January 23, 2023 17:02
@glimchb glimchb requested a review from PWAlessi January 23, 2023 17:03
@codecov
Copy link

codecov bot commented Jan 23, 2023

Codecov Report

Merging #26 (dfe7eb8) into main (9484f76) will increase coverage by 23.70%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main      #26       +/-   ##
===========================================
+ Coverage   36.70%   60.41%   +23.70%     
===========================================
  Files           2        1        -1     
  Lines          79       48       -31     
===========================================
  Hits           29       29               
+ Misses         44       13       -31     
  Partials        6        6               
Impacted Files Coverage Δ
pkg/inventory/inventory.go 60.41% <ø> (ø)
cmd/main.go

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Following https://github.com/golang-standards/project-layout

Signed-off-by: Boris Glimcher <Boris.Glimcher@emc.com>
Signed-off-by: Boris Glimcher <Boris.Glimcher@emc.com>
@glimchb glimchb merged commit 49982ac into main Jan 23, 2023
@glimchb glimchb deleted the docker branch January 23, 2023 17:43
@glimchb glimchb changed the title Folder refactor Folder refactor and new CI compose test Jan 23, 2023
Copy link

@PWAlessi PWAlessi left a comment

Choose a reason for hiding this comment

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

LGTM

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants