Skip to content
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

#1 Telemetry update #311

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

#1 Telemetry update #311

wants to merge 13 commits into from

Conversation

allanchau
Copy link
Member

@allanchau allanchau commented Dec 5, 2022

This PR cleans up the @idearium/telemetry packages and updates them to the latest version.

Related PRs

Notes

  • Express was removed (not sure why that was included, but it wasn't being required).
  • @opentelemetry/core was added to stop an install warning about missing peer dependencies.
  • apm was updated to fix a build error in github actions.
  • This doesn't seem to work, it causes the app to stop bootstrapping.

Verification and testing

Testing

  • Put this in a projects, it should work exactly the same as before.

Deployment

  • Release a new version of @idearium/log.
  • Update cookie to use the new @idearium/log.
  • Update @idearium/log-http to use the new @idearium/log.
  • Update @idearium/mongoose to use the new @idearium/log.
  • Release a new Telemetry package version.
  • Update @idearium/log in log-http and release a new version.
  • Update @idearium/log in log-structured and release a new version.

@allanchau allanchau requested a review from a team December 5, 2022 00:42
@allanchau allanchau self-assigned this Dec 5, 2022
@allanchau allanchau marked this pull request as ready for review December 5, 2022 00:42
@allanchau allanchau changed the title Telemetry update #1 Telemetry update Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant