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

Feature/telemetry 1 #13

Merged
merged 7 commits into from
Sep 19, 2024
Merged

Feature/telemetry 1 #13

merged 7 commits into from
Sep 19, 2024

Conversation

sroy9675
Copy link
Contributor

@sroy9675 sroy9675 commented Aug 23, 2024

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Documentation or Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes

Description

Adding basic telemetry collection and reporting. Update environment to provide deployment and api version information. Report this info along with uuid to telemetry endpoint

Changes

Issue Number (if applicable):

Tests

99% test coverage

Other information

Copy link

codecov bot commented Aug 23, 2024

Codecov Report

Attention: Patch coverage is 99.75904% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
app/test_telemetry.py 99.67% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@sroy9675 sroy9675 requested a review from pselvana August 26, 2024 19:21
Copy link
Contributor

@pselvana pselvana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sroy9675 sroy9675 merged commit 7b76e4c into dev Sep 19, 2024
3 checks passed
@sroy9675 sroy9675 deleted the feature/telemetry_1 branch September 19, 2024 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants