Conversation
17444b3 to
cc14971
Compare
|
✨ Fix all issues with BitsAI or with Cursor
|
| end | ||
|
|
||
| # Minimal stats tracker matching the interface used by Writer#stats. | ||
| class Stats |
There was a problem hiding this comment.
Are stats getting incremented anywhere?
| @@ -0,0 +1,317 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
Does this get automatically run with the test suite on this PR? Would this eventually become a Rakefile? Or is your intention to keep it a bash script?
There was a problem hiding this comment.
This is a helper script for quick iteration.
I certainly don't aim it to be merged (at least not as is).
If it needs to be kept it should be turned into rake tasks.
Ideally I'd like the workflow to be able to evolve and dd-trace-rb's rake compile to properly rebuild without shenanigans (the current rake compile has trouble with stale files and such)
Or even better: the ext folder could move to libdatadog-rb and dd-trace-rb be free of C extensions? That would allow libdatadog-rb to compile and build end-to-end, possibly maximising LTO savings.
Probably not for this task though.
cc14971 to
dac05cf
Compare
What does this PR do?
Add:
Motivation:
Getting the ball rolling on discussion with actual code
Change log entry
None
Additional Notes:
DO NOT MERGE
How to test the change?
Locally:
libdatadog/ruby: runrake compiledd-trace-rb: appendgem 'libdatadog', path: '../libdatadog/ruby'toruby-X.Y.gemfiledd-trace-rb: runbash compile_and_test.shSee DataDog/libdatadog#1661