Thanks for the suggestion, but this library is .Net Standard 2.0, and that compiler directive will never be triggered.

I actually just had a conversation on Twitter regarding things like this that make the code harder to maintain. I prefer to avoid them.

and that compiler directive will never be triggered.

OK, I just found #460 which seems to indicate that you were having plans to upgrade or multi target at some point.

Upgrade eventually, yes, but not multi-target. As it stands, I apparently have some consumers at Microsoft that are still using Framework, so moving off of Standard probably won't be for a bit (or at least until I can articulate a convincing reason to get them to update our leave them behind).

I'm going to close this for now, but I'll definitely make these kinds of updates in the future. Thanks for the suggestion.

@ivarne it looks like I might be moving toward updating sooner than I expected. The demand for Native AOT support is growing. If I need to do that I might as well do this as well.

See #618 for progress.

@ivarne I've retargeted this PR to native-aot, where this would be useful. Yeah, I could just add the attribute myself, but I thought you'd like to have the commit.

Thanks!

This isn't my first PR, but if it was it would have mattered a lot.