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

Support stats "yjit_alloc_size" #3692

Closed
wants to merge 3 commits into from

Conversation

p-datadog
Copy link
Contributor

Run full CI for #3661

MITSUBOSHI and others added 3 commits May 23, 2024 21:15
lib/datadog/core/runtime/metrics.rb:143:9: C: Metrics/MethodLength: Method has too many lines. [40/36]
        def flush_yjit_stats ...
        ^^^^^^^^^^^^^^^^^^^^
* master: (34 commits)
  Add hanami app to integration CI configurations (#3639)
  Update UpgradeGuide in README.md
  Update lockfiles for release 2.0.0
  Bump version 2.0.0.rc1 to 2.0.0
  Add 2.0.0 to CHANGELOG.md
  Add missing Apache 2.0 license from gemspec
  Use ffi 1.16.3
  Add platform
  Output gem version
  Output debug logs
  Fix spec to exclude .vscode directory
  Fix docker run command
  Add vscode settings for dockerfile file associations
  Update gem version
  Update gemfiles/*
  Update `ffi` in Gemfile
  Remove experimental warning from OpenTelemetry.md
  Remove latest tag on docker images
  add installer system tests
  Extract trace id format helper method (#3670)
  ...
@p-datadog p-datadog requested a review from a team as a code owner June 6, 2024 21:04
@github-actions github-actions bot added the core Involves Datadog core libraries label Jun 6, 2024
@MITSUBOSHI
Copy link
Contributor

@p-datadog

Thank you for checking my PR.

I pushed two changes to #3661 in order to pass CI.

@p-datadog p-datadog closed this Jun 12, 2024
@p-datadog p-datadog deleted the feature/add-more-metrics-for-yjit branch June 12, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Involves Datadog core libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants