Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Removed serde_derive dependency #279

Merged
merged 1 commit into from
Aug 12, 2021
Merged

Removed serde_derive dependency #279

merged 1 commit into from
Aug 12, 2021

Conversation

jorgecarleitao
Copy link
Owner

Moved file around, which allowed to remove the dependency that is only used for testing.

@codecov
Copy link

codecov bot commented Aug 12, 2021

Codecov Report

Merging #279 (818cd46) into main (c7e1a4d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #279   +/-   ##
=======================================
  Coverage   77.15%   77.15%           
=======================================
  Files         254      254           
  Lines       20649    20649           
=======================================
  Hits        15931    15931           
  Misses       4718     4718           
Impacted Files Coverage Δ
src/io/json/mod.rs 100.00% <ø> (ø)
src/io/json_integration/mod.rs 0.00% <ø> (ø)
src/io/json_integration/schema.rs 42.12% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c7e1a4d...818cd46. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit d1a5025 into main Aug 12, 2021
@jorgecarleitao jorgecarleitao deleted the no_derive branch August 12, 2021 21:51
@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Aug 12, 2021
@jorgecarleitao jorgecarleitao changed the title Removed serde_derive dependency. Removed serde_derive dependency Aug 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant