Thomas Watson

Results 68 issues of Thomas Watson

### What does this PR do? Relying on string length to calculate the number of bytes in the string is incorrect as the `length` property returns number of characters -...

semver-patch

### What does this PR do? Configure supported Node.js version range using [repository variables](https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables) Currently, the following repository variables are created: - `ACTIVE_LTS` (set to `22` as of this commit)...

semver-patch

### What does this PR do? This runs _all_ system tests instead of just a subset and tries to balance them out as much as possible, to get below the...

semver-patch

### What does this PR do? Use a WeakMap to store the Code Origin Entry Span location instead of polluting the Fastify route options config object. Also limit when the...

semver-patch
integration-fastify

### What does this PR do? Don't wait for script loading to stabilize in Dynamic Instrumentation integration tests. This might make them temporarily set the test breakpoint in the wrong...

semver-patch

### What does this PR do? Removes a polyfill used to support Node.js versions below version 19.0.0. ### Motivation For the upcoming release of dd-trace version 6, these Node.js versions...

breaking-change
semver-major
dynamic-instrumentation

### What does this PR do? Enables [all recommended rules](https://github.com/eslint-community/eslint-plugin-n?tab=readme-ov-file#-rules) from the `eslint-plugin-n` ESLint plugin that could reasonably be enabled without too much work. The following rules have for now...

semver-patch

### What does this PR do? The target thread is paused for some amount of time when ever a breakpoint is hit. Record the amount of time in milliseconds as...

semver-patch
dynamic-instrumentation

### What does this PR do? Delete `.npmignore` and use the `files` property in `package.json` to select the files to include in the published npm package. ### Motivation The `n/no-unpublished-require`...

semver-patch

### What does this PR do? Require [strict mode](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode) on all CommonJS modules.

semver-patch