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

Review sampling docs for Ruby [v2] #1958

Merged
merged 3 commits into from
Apr 4, 2022

Conversation

dussault-antoine
Copy link
Contributor

Remove Tracing without Limits occurrences from the Ruby tracer documentation

@dussault-antoine dussault-antoine requested a review from a team April 4, 2022 09:25
Copy link
Member

@ivoanjo ivoanjo left a comment

Choose a reason for hiding this comment

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

Left two suggestions :)

@@ -2068,21 +2068,11 @@ Datadog.configure { |c| c.diagnostics.startup_logs.enabled = true }

By default, this will be activated whenever `ddtrace` detects the application is running in a non-development environment.

### Sampling
Copy link
Member

Choose a reason for hiding this comment

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

I think we should keep this heading, as the two next sections (application-side sampling, priority sampling) are supposed to be under it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added back 👍

docs/GettingStarted.md Outdated Show resolved Hide resolved
dussault-antoine and others added 2 commits April 4, 2022 11:32
Co-authored-by: Ivo Anjo <ivo.anjo@datadoghq.com>
Copy link
Member

@ivoanjo ivoanjo left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@codecov-commenter
Copy link

Codecov Report

Merging #1958 (4c39463) into master (027aa58) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1958   +/-   ##
=======================================
  Coverage   97.53%   97.53%           
=======================================
  Files         998      998           
  Lines       49006    49006           
=======================================
  Hits        47800    47800           
  Misses       1206     1206           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@ivoanjo ivoanjo merged commit 60ecc8a into master Apr 4, 2022
@ivoanjo ivoanjo deleted the antoine.dussault/remove_twl_occurrences branch April 4, 2022 10:29
@github-actions github-actions bot added this to the 1.0.0.beta2 milestone Apr 4, 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