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

feat: add IsDouble function #29076

Conversation

etiennep
Copy link
Contributor

@etiennep etiennep commented Nov 8, 2023

Description: Adds new a new IsDouble function to facilitate type checking. Most useful when checking the type of a body to determine if it needs to be parsed or not.

Link to tracking Issue: #27895

Testing: Added unit test

Documentation: Updated the func readme.

Copy link

linux-foundation-easycla bot commented Nov 8, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: etiennep / name: Etienne Pelletier (af22f14)
  • ✅ login: TylerHelmuth / name: Tyler Helmuth (8da35ae)

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Thanks @etiennep! Please sign the CLA

@atoulme
Copy link
Contributor

atoulme commented Nov 8, 2023

Please run make chlog-new from the root of the repository to generate a changelog entry for this contribution, fill it up with issue information and add it to the PR.

@etiennep etiennep force-pushed the etiennep.issue-27895-IsDouble-converter branch 3 times, most recently from 70d567d to 372f1e4 Compare November 8, 2023 23:15
@etiennep etiennep force-pushed the etiennep.issue-27895-IsDouble-converter branch from 372f1e4 to a546ea3 Compare November 9, 2023 15:47
@etiennep etiennep force-pushed the etiennep.issue-27895-IsDouble-converter branch from a546ea3 to 3760866 Compare November 9, 2023 23:40
@etiennep etiennep force-pushed the etiennep.issue-27895-IsDouble-converter branch 4 times, most recently from 847c497 to a33c1d5 Compare November 12, 2023 22:24
@TylerHelmuth
Copy link
Member

@etiennep I merged another function so we need a rebase.

@etiennep etiennep force-pushed the etiennep.issue-27895-IsDouble-converter branch from a33c1d5 to af22f14 Compare November 15, 2023 03:06
@etiennep
Copy link
Contributor Author

@etiennep I merged another function so we need a rebase.

Done!

@TylerHelmuth TylerHelmuth added the ready to merge Code review completed; ready to merge by maintainers label Nov 16, 2023
@TylerHelmuth TylerHelmuth merged commit 533b72d into open-telemetry:main Nov 16, 2023
87 of 88 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 16, 2023
graphaelli pushed a commit to graphaelli/opentelemetry-collector-contrib that referenced this pull request Nov 17, 2023
**Description:** Adds new a new `IsDouble` function to facilitate type
checking. Most useful when checking the type of a body to determine if
it needs to be parsed or not.

**Link to tracking Issue:**
open-telemetry#27895

**Testing:** Added unit test

**Documentation:** Updated the func readme.

Co-authored-by: Tyler Helmuth <[email protected]>
RoryCrispin pushed a commit to ClickHouse/opentelemetry-collector-contrib that referenced this pull request Nov 24, 2023
**Description:** Adds new a new `IsDouble` function to facilitate type
checking. Most useful when checking the type of a body to determine if
it needs to be parsed or not.

**Link to tracking Issue:**
open-telemetry#27895

**Testing:** Added unit test

**Documentation:** Updated the func readme.

Co-authored-by: Tyler Helmuth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribfest pkg/ottl ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants