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

Un-nested compute::arithemtics::basic #461

Merged
merged 1 commit into from
Sep 29, 2021
Merged

Un-nested compute::arithemtics::basic #461

merged 1 commit into from
Sep 29, 2021

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Sep 29, 2021

This PR removes the modules compute::arithmetics::basic::* from the public API, instead re-exporting all functions on them to compute::arithmetics::basic, making them a bit easier to find and use.

To migrate, replace compute::arithmetics::basic::add, compute::arithmetics::basic::sub, etc. by compute::arithmetics::basic.

@jorgecarleitao jorgecarleitao force-pushed the unested branch 2 times, most recently from 32d8711 to 43a17c5 Compare September 29, 2021 05:44
@jorgecarleitao jorgecarleitao changed the title Removed one nest. Un-nested compute::arithemtics::basic Sep 29, 2021
@jorgecarleitao jorgecarleitao force-pushed the unested branch 2 times, most recently from 7f3b23d to 1952ddb Compare September 29, 2021 05:54
@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

Merging #461 (b28fd99) into main (fedb19f) will increase coverage by 0.00%.
The diff coverage is 83.33%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #461   +/-   ##
=======================================
  Coverage   80.03%   80.03%           
=======================================
  Files         371      371           
  Lines       22839    22839           
=======================================
+ Hits        18279    18280    +1     
+ Misses       4560     4559    -1     
Impacted Files Coverage Δ
src/compute/arithmetics/basic/add.rs 85.18% <ø> (ø)
src/compute/arithmetics/basic/div.rs 81.66% <ø> (ø)
src/compute/arithmetics/basic/mul.rs 85.18% <ø> (ø)
src/compute/arithmetics/basic/pow.rs 100.00% <ø> (ø)
src/compute/arithmetics/basic/rem.rs 92.30% <ø> (ø)
src/compute/arithmetics/basic/sub.rs 85.18% <ø> (ø)
tests/it/compute/arithmetics/basic/add.rs 100.00% <ø> (ø)
tests/it/compute/arithmetics/basic/div.rs 100.00% <ø> (ø)
tests/it/compute/arithmetics/basic/mul.rs 100.00% <ø> (ø)
tests/it/compute/arithmetics/basic/pow.rs 100.00% <ø> (ø)
... and 6 more

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 fedb19f...b28fd99. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant