Skip to content

otel: capture exit code as int64#4985

Merged
laurazard merged 1 commit intodocker:masterfrom
laurazard:otel-exit-code-int
Apr 5, 2024
Merged

otel: capture exit code as int64#4985
laurazard merged 1 commit intodocker:masterfrom
laurazard:otel-exit-code-int

Conversation

@laurazard
Copy link
Copy Markdown
Member

- What I did

Capture exit code as int64 (it used to be a string!)

- How I did it

By changing characters in source files!

- How to verify it

Run the relevant tests.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@laurazard laurazard requested a review from jsternberg April 4, 2024 18:16
@laurazard laurazard self-assigned this Apr 4, 2024
Copy link
Copy Markdown
Contributor

@jsternberg jsternberg left a comment

Choose a reason for hiding this comment

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

Yea I think this is much better.

@laurazard laurazard merged commit b2fe82a into docker:master Apr 5, 2024
@neersighted neersighted deleted the otel-exit-code-int branch April 5, 2024 19:29
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.03%. Comparing base (10b9810) to head (cefcba9).
Report is 553 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4985   +/-   ##
=======================================
  Coverage   61.03%   61.03%           
=======================================
  Files         295      295           
  Lines       20648    20648           
=======================================
  Hits        12603    12603           
  Misses       7148     7148           
  Partials      897      897           

@thaJeztah thaJeztah added this to the 26.1.0 milestone Sep 18, 2024
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.

5 participants