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

[WIP]update check docker compose #21076

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MinerYang
Copy link
Contributor

Thank you for contributing to Harbor!

Comprehensive Summary of your change

Issue being fixed

Fixes #(issue)

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

Signed-off-by: yminer <miner.yang@broadcom.com>
@MinerYang MinerYang added the release-note/ignore-for-release Do not include PR or Issue for release notes label Oct 22, 2024
@MinerYang MinerYang requested a review from a team as a code owner October 22, 2024 09:55
Copy link

codecov bot commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.09%. Comparing base (c8c11b4) to head (c9a0cb1).
Report is 341 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #21076       +/-   ##
===========================================
+ Coverage   45.36%   66.09%   +20.72%     
===========================================
  Files         244     1049      +805     
  Lines       13333   114645   +101312     
  Branches     2719     2867      +148     
===========================================
+ Hits         6049    75771    +69722     
- Misses       6983    34735    +27752     
- Partials      301     4139     +3838     
Flag Coverage Δ
unittests 66.09% <ø> (+20.72%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1288 files with indirect coverage changes

@MinerYang MinerYang changed the title update check docker compose [WIP]update check docker compose Oct 22, 2024
@@ -103,7 +103,7 @@ function check_docker {
}

function check_dockercompose {
if [! docker compose version] &> /dev/null || [! docker-compose --version] &> /dev/null
if ! [docker compose version &> /dev/null] || ! [docker-compose --version &> /dev/null]
Copy link

Choose a reason for hiding this comment

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

Suggested change
if ! [docker compose version &> /dev/null] || ! [docker-compose --version &> /dev/null]
if ! docker compose version &> /dev/null || ! docker-compose --version &> /dev/null

Copy link

This PR is being marked stale due to a period of inactivty. If this PR is still relevant, please comment or remove the stale label. Otherwise, this PR will close in 30 days.

@github-actions github-actions bot added the Stale label Dec 25, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
release-note/ignore-for-release Do not include PR or Issue for release notes Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants