Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Adds missing dash to --build-arg #1559

Merged
merged 1 commit into from May 29, 2022

Conversation

elektro-wolle
Copy link
Contributor

According to the help-output of docker buildx build --help:

Usage:  docker buildx build [OPTIONS] PATH | URL | -
[...]
Options:
      --add-host strings              Add a custom host-to-IP mapping (format: "host:ip")
      --allow strings                 Allow extra privileged entitlement (e.g., "network.host", "security.insecure")
      --build-arg stringArray         Set build-time variables
[...]

@rohanKanojia
Copy link
Member

@chonton : Could you please review this whenever you get time?

@chonton
Copy link
Contributor

chonton commented May 25, 2022

looks good. how about a unit test?

According to the help-output of `docker buildx build --help`:

```
Usage:  docker buildx build [OPTIONS] PATH | URL | -
[...]
Options:
      --add-host strings              Add a custom host-to-IP mapping (format: "host:ip")
      --allow strings                 Allow extra privileged entitlement (e.g., "network.host", "security.insecure")
      --build-arg stringArray         Set build-time variables
[...]
```
@sonarcloud
Copy link

sonarcloud bot commented May 29, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented May 29, 2022

Codecov Report

Merging #1559 (5b319fc) into master (f898132) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1559      +/-   ##
============================================
+ Coverage     62.65%   62.70%   +0.05%     
- Complexity     2141     2143       +2     
============================================
  Files           170      170              
  Lines          9886     9886              
  Branches       1355     1355              
============================================
+ Hits           6194     6199       +5     
+ Misses         3176     3172       -4     
+ Partials        516      515       -1     
Impacted Files Coverage Δ
...io/fabric8/maven/docker/service/BuildXService.java 65.62% <100.00%> (+3.90%) ⬆️

@rohanKanojia rohanKanojia merged commit a433749 into fabric8io:master May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants