Skip to content

Conversation

@neSpecc
Copy link
Member

@neSpecc neSpecc commented Oct 15, 2025

  • Log every mongo request
  • Log graphql operations
  • Fix redundant "releases" call when event has no release
  • Removed unused "deepMerge" util
image

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 87.03704% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.14%. Comparing base (0eced98) to head (81cc005).
⚠️ Report is 59 commits behind head on master.

Files with missing lines Patch % Lines
src/metrics/mongodb.ts 74.07% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #555      +/-   ##
==========================================
- Coverage   66.66%   61.14%   -5.53%     
==========================================
  Files          11       19       +8     
  Lines         174      471     +297     
  Branches       24       92      +68     
==========================================
+ Hits          116      288     +172     
- Misses         54      171     +117     
- Partials        4       12       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@neSpecc neSpecc merged commit 5f1a5bf into master Oct 15, 2025
6 checks passed
@neSpecc neSpecc deleted the logs branch October 15, 2025 20:35
neSpecc added a commit that referenced this pull request Oct 15, 2025
* chore(logs): improve mongo and graphql logs

* beautify logs

* Bump version up to 1.2.5

* rm utils

* improve grapgql log

* improve mongo logs

* Update logger.ts

---------

Co-authored-by: Peter <[email protected]>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

@fdtwd8vv45-sketch fdtwd8vv45-sketch left a comment

Choose a reason for hiding this comment

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

Looks good

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.

6 participants