Skip to content

[tool] fix(deprecation): move pythonjsonlogger.jsonlogger to pythonjsonlogger.json (#212)#213

Merged
ncarenton merged 3 commits intomainfrom
fix/212-josn-deprecatin
May 4, 2026
Merged

[tool] fix(deprecation): move pythonjsonlogger.jsonlogger to pythonjsonlogger.json (#212)#213
ncarenton merged 3 commits intomainfrom
fix/212-josn-deprecatin

Conversation

@mariot
Copy link
Copy Markdown
Member

@mariot mariot commented Apr 28, 2026

Proposed changes

Testing Instructions

  1. Step-by-step how to test
  2. Environment or config notes

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

@mariot mariot requested review from Kakudou, guzmud and ncarenton April 28, 2026 08:37
@mariot mariot self-assigned this Apr 28, 2026
@mariot mariot added dependencies use for pull requests that update a dependency file filigran team use to identify PR from the Filigran team tiny use to identify tiny feature to develop or bug to fix (< a week to solve) labels Apr 28, 2026
@mariot mariot force-pushed the fix/212-josn-deprecatin branch from fedcfd3 to 09cc802 Compare April 28, 2026 08:40
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.06%. Comparing base (58abf4c) to head (84caa92).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #213      +/-   ##
==========================================
- Coverage   70.41%   70.06%   -0.35%     
==========================================
  Files          58       49       -9     
  Lines        2329     1941     -388     
==========================================
- Hits         1640     1360     -280     
+ Misses        689      581     -108     
Flag Coverage Δ
connectors 70.06% <100.00%> (-0.35%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ncarenton ncarenton merged commit ff61d2f into main May 4, 2026
12 checks passed
@ncarenton ncarenton deleted the fix/212-josn-deprecatin branch May 4, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies use for pull requests that update a dependency file filigran team use to identify PR from the Filigran team tiny use to identify tiny feature to develop or bug to fix (< a week to solve)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pythonjsonlogger.jsonlogger is deprecated and has been moved to pythonjsonlogger.json

2 participants