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
Yes, perhaps indeed. However, it seems too late to change anything here. This is exactly the naming conflict that namespaces exist to work around. Or do you propose some specific change to the library with this issue? It is not clear what is the purpose of this post - please clarify.
It was mostly to point out the issue as most of the people using the library will face the issue upgrading to .NET8+ considering how common System.Threading namespace is.
The fix is easy enough and just require to convert most variable declaration from ITimer to Prometheus.ITimer.
So, if you think changing the interface name is much worse than that (and I think you are probably right) you can archive the issue.
.NET8 introduce an new
System.Threading.ITimer
interface that have the same name asPrometheus.ITimer
You need to change all
ITimer
toPrometheus.ITimer
to fix the .NET8 upgrade.Maybe
ITimer
is a little too generic of a name no?The text was updated successfully, but these errors were encountered: