-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add coverage for application insights #20828
Comments
Hi @maorleger, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support. |
Consider issues that only occur when application-insights is enabled such as:
Since we do not include applicationinsights in our smoke tests, integration tests, etc. we have missed some compatibility issues leading to costly investigations from both teams.
This leads me to believe that we will benefit from adding applicationinsights to our tests. This issue tracks investigating what would be the right approach here.
A few thoughts:
The text was updated successfully, but these errors were encountered: