Skip to content

Conversation

p-rogalski and others added 3 commits November 13, 2024 23:36
Signed-off-by: Paul Rogalski <[email protected]>
(cherry picked from commit c70b216)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
The command to run inside the container is `/etc`. The semicolon is a
statement terminator, which ends the command `docker run busybox /etc`,
while `echo $?` prints the exit code of that full docker command.

Having this mistake could confuse someone who thinks that `/etc; echo
$?` is all run inside the container, which wouldn't help the reader
understand the exit code of the `docker run` command itself.

Signed-off-by: Noah Silas <[email protected]>
(cherry picked from commit 0c999fe)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: David Karlsson <[email protected]>
(cherry picked from commit 172f340)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 27.4.0 milestone Nov 13, 2024
@thaJeztah thaJeztah self-assigned this Nov 13, 2024
@thaJeztah thaJeztah requested a review from a team as a code owner November 13, 2024 22:38
@thaJeztah thaJeztah requested a review from a team November 13, 2024 22:39
@codecov-commenter
Copy link

codecov-commenter commented Nov 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.37%. Comparing base (e15a979) to head (2734299).
Report is 11 commits behind head on 27.x.

Additional details and impacted files
@@            Coverage Diff             @@
##             27.x    #5612      +/-   ##
==========================================
- Coverage   58.65%   58.37%   -0.28%     
==========================================
  Files         345      336       -9     
  Lines       29053    28781     -272     
==========================================
- Hits        17040    16800     -240     
+ Misses      11042    11013      -29     
+ Partials      971      968       -3     

@thaJeztah thaJeztah merged commit d5b3272 into docker:27.x Nov 14, 2024
99 checks passed
@thaJeztah thaJeztah deleted the 27.x_backport_docs branch November 14, 2024 09:42
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.

6 participants