-
-
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
Better Surface SDK Examples Repo #3530
Comments
We should validate all of the examples in there before we do this, probably. Just checked it out and a few of them haven't been touched in a long time. (Also might be worth having a conversation about maintenance expectations in general. Right now I don't think this is on most people's radar as something that needs to be kept in sync with changes in the SDK. cc @HazAT @rhcarvalho @bruno-garcia ) To be clear - I think having examples is great, and even better if people know they exist. I am all for this issue in general. But pointing people to ones that no longer work feels worse than them being hidden away. |
Makes sense to verify examples are up-to-date. One approach is to use the release registry to explicitly link to examples as suggested here: getsentry/sentry-release-registry#39 (comment) This way we'd be explicitly declaring an example (or more?) for a package, and if there are outdated ones on that repo it shouldn't matter as much |
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
As noted on Twitter, we need to better surface that we offer an examples repo for many of our SDKs.
We have this listed in the primer for some of the mobile SDKS, but it should become part of our standard list if it's available.
The text was updated successfully, but these errors were encountered: