Add `dispatch-value-spec`/`defmethod-arities` to multimethods without them #157

camsaul · 2023-10-03T21:30:57Z

Resolves #99

…n't have them

codecov · 2023-10-03T21:32:06Z

Codecov Report

Attention: 21 lines in your changes are missing coverage. Please review.

Files	Coverage Δ
src/toucan2/types.clj	`100.00% <100.00%> (ø)`
src/toucan2/jdbc/result_set.clj	`85.18% <0.00%> (+1.03%)`	⬆️
src/toucan2/pipeline.clj	`77.01% <0.00%> (ø)`
src/toucan2/save.clj	`70.27% <33.33%> (-3.26%)`	⬇️
src/toucan2/tools/after.clj	`76.47% <0.00%> (+0.35%)`	⬆️
src/toucan2/tools/after_select.clj	`86.95% <33.33%> (-3.96%)`	⬇️
src/toucan2/query.clj	`85.83% <0.00%> (ø)`
src/toucan2/jdbc/read.clj	`75.00% <20.00%> (-2.95%)`	⬇️
src/toucan2/model.clj	`81.17% <0.00%> (-2.97%)`	⬇️

📢 Thoughts on this report? Let us know!.

Add dispatch-value-spec/defmethod-arities to multimethods that do…

2e8b4aa

…n't have them

camsaul enabled auto-merge (squash) October 3, 2023 21:31

camsaul disabled auto-merge October 3, 2023 21:32

camsaul merged commit 0cb6996 into master Oct 3, 2023
9 of 10 checks passed

camsaul deleted the add-dispatch-value-spec-and-defmethod-arities branch October 3, 2023 21:32