Andrew Lock

Results 78 issues of Andrew Lock

## Summary of changes - Adds explicit permissions to all workflows ## Reason for change It's a best practice to explicitly specify required permissions for all workflows, and soon we'll...

area:builds

## Summary of changes Fixes `IsServerBuild` calculation in CI ## Reason for change It wasn't working correctly, e.g. [here](https://dev.azure.com/datadoghq/dd-trace-dotnet/_build/results?buildId=160073&view=logs&j=4d2bd072-69d2-52f9-9ef7-1ebf54e62e75&t=932c758b-5f44-5d34-a86c-84ce5d520f99&l=6417) ## Implementation details Passes the `TF_BUILD` env var through to docker...

area:builds

> [!IMPORTANT] > This PR should be rebased onto `release/2.x` once that branch is created. It should _not_ be merged into `master` ## Summary of changes Removes the "deploy latest"...

area:builds
status:do-not-merge

## Motivation ## Changes ## Workflow 1. ⚠️ Create your PR as draft ⚠️ 2. Work on you PR until the CI passes (if something not related to your task...

## Summary of changes - Adds a test that uses manual instrumentation with Ready-to-run (r2r) - Disable using the ngen images of Datadog.Trace.Manual ## Reason for change In #6124 we...

type:bug
area:tracer
area:automatic-instrumentation
area:native-library

## Summary of changes Adds a check that IisPreStartInit has completed before we run any automatic instrumentations ## Reason for change We recently had a case where a customer was...

type:bug
area:automatic-instrumentation
status:do-not-merge

## Summary of changes Before injecting the startup hook, check if the type has an explicit static constructor. If so, skip injecting. ## Reason for change This fixes an issue...

## Summary of changes Runs additional installer tests for alpine where the glibc compatibility package is installed ## Reason for change It's possible to run glibc libraries on alpine if...

area:builds
area:tests

If possible (and available), we should use newer C# features in the generated code. e.g. - CollectionExpressions (C#12+) - ReadOnlySpan (currently `#if` by platform, we _could_ consider polyfilling? Or adding...