You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The usage of TypeAttributes.Serializable seems to have been used since its introduction into the Datadog automatic instrumentation. I've messaged my colleagues to understand if it's truly needed, and if it is then we'll need to suppress this compiler warning. Or perhaps if it's not needed for newer runtimes we can ifdef its usage, but I don't know if that's the case
I asked around with some colleagues at Datadog and the consensus is that there is risk in removing the attribute, but there's no clear scenario where the error could occur. To be safe, the recommendation is to keep applying the Serializable attribute on the generated types (keeping the code as-is).
Originally posted by @zacharycmontoya in #3796 (comment)
It can be done after merge, it was working fine on .NET8 with .NET6 binaries for a while. No rush to changes.
The text was updated successfully, but these errors were encountered: