Skip to content

🔥 Remove orjson#92

Merged
aleksul merged 1 commit intomainfrom
ditch-orjson
Apr 30, 2023
Merged

🔥 Remove orjson#92
aleksul merged 1 commit intomainfrom
ditch-orjson

Conversation

@aleksul
Copy link
Copy Markdown
Owner

@aleksul aleksul commented Apr 29, 2023

Change Summary

Remove orjson in favor of standard library json.

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • My PR is ready to review

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 29, 2023

Codecov Report

Merging #92 (4033b62) into main (41e7618) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #92   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           43        43           
  Lines         1676      1669    -7     
=========================================
- Hits          1676      1669    -7     
Impacted Files Coverage Δ
repid/_processor.py 100.00% <100.00%> (ø)
repid/connections/rabbitmq/consumer.py 100.00% <100.00%> (ø)
repid/connections/rabbitmq/message_broker.py 100.00% <100.00%> (ø)
repid/converter.py 100.00% <100.00%> (ø)
repid/data/_buckets.py 100.00% <100.00%> (ø)
repid/data/_parameters.py 100.00% <100.00%> (ø)
repid/job.py 100.00% <100.00%> (ø)
repid/serializer.py 100.00% <100.00%> (ø)
repid/utils.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@aleksul aleksul merged commit 2689072 into main Apr 30, 2023
@aleksul aleksul deleted the ditch-orjson branch April 30, 2023 00:20
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.

1 participant