Skip to content

Use consistent syntax in profiling vignette#1137

Merged
jgabry merged 1 commit intomasterfrom
fix-profiling-vignette
Feb 11, 2026
Merged

Use consistent syntax in profiling vignette#1137
jgabry merged 1 commit intomasterfrom
fix-profiling-vignette

Conversation

@jgabry
Copy link
Member

@jgabry jgabry commented Feb 9, 2026

closes #1136

Submission Checklist

  • Run unit tests
  • Declare copyright holder and agree to license (see below)

Summary

Don't switch between target += and ~ syntax.

Copyright and Licensing

Please list the copyright holder for the work you are submitting
(this will be you or your assignee, such as a university or company):
Jonah Gabry

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the following licenses:

@jgabry jgabry added the documentation Improvements or additions to documentation label Feb 9, 2026
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.46%. Comparing base (c062d6c) to head (601e59e).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1137      +/-   ##
==========================================
- Coverage   87.49%   86.46%   -1.04%     
==========================================
  Files          14       14              
  Lines        5983     5983              
==========================================
- Hits         5235     5173      -62     
- Misses        748      810      +62     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

1 similar comment
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.46%. Comparing base (c062d6c) to head (601e59e).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1137      +/-   ##
==========================================
- Coverage   87.49%   86.46%   -1.04%     
==========================================
  Files          14       14              
  Lines        5983     5983              
==========================================
- Hits         5235     5173      -62     
- Misses        748      810      +62     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jgabry
Copy link
Member Author

jgabry commented Feb 11, 2026

The failure on R-devel is unrelated to this PR. Merging now.

@jgabry jgabry merged commit 453084b into master Feb 11, 2026
31 of 36 checks passed
@jgabry jgabry deleted the fix-profiling-vignette branch February 11, 2026 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation bug in profiling vignette

2 participants