-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Removed metrics documentation for .NET #11880
base: master
Are you sure you want to change the base?
Removed metrics documentation for .NET #11880
Conversation
@jamescrosswell is attempting to deploy a commit to the Sentry Team on Vercel. A member of the Team first needs to authorize it. |
I see, the linting error implies this can't be removed for .NET in isolation (it would need to be removed for all the other platforms as well presumably?):
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
@bitsandfoxes can you take a look? |
We can also remove the reference to the deleted page here
|
Yeah I assumed that was the language agnostic docs though so would affect all the SDKs (not just the .NET SDK)... and I didn't want to mess with that. |
I think it's fine to remove the entry on the product page as it links specifically to the page that' getting removed. |
DESCRIBE YOUR PR
Removing Metrics documentation now this feature has been deprecated
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
@bruno-garcia you're the only person that I can assign as a reviewer on this sorry... I can't select Bryce in this repo.
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes: