Skip to content

Conversation

@Luni-4
Copy link
Collaborator

@Luni-4 Luni-4 commented Nov 24, 2021

This PR removes a useless macro from the CLI making the code more readable

@codecov-commenter
Copy link

Codecov Report

Merging #708 (04dcbd9) into master (a507c7d) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #708   +/-   ##
=======================================
  Coverage   37.16%   37.16%           
=======================================
  Files          50       50           
  Lines        6294     6283   -11     
  Branches      936      937    +1     
=======================================
- Hits         2339     2335    -4     
+ Misses       3311     3304    -7     
  Partials      644      644           
Impacted Files Coverage Δ
rust-code-analysis-cli/src/formats.rs 0.00% <0.00%> (ø)
rust-code-analysis-cli/src/main.rs 0.34% <0.00%> (+<0.01%) ⬆️
src/comment_rm.rs 60.00% <0.00%> (-0.79%) ⬇️
src/spaces.rs 64.45% <0.00%> (-0.22%) ⬇️
src/ops.rs 82.95% <0.00%> (-0.08%) ⬇️
src/find.rs 0.00% <0.00%> (ø)
src/preproc.rs 0.00% <0.00%> (ø)
src/function.rs 20.00% <0.00%> (+0.35%) ⬆️
src/node.rs 71.15% <0.00%> (+0.78%) ⬆️

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 a507c7d...04dcbd9. Read the comment docs.

@Luni-4 Luni-4 force-pushed the remove-macro branch 2 times, most recently from b4b098d to 58fe0f9 Compare November 26, 2021 14:30
@marco-c marco-c merged commit 86614b4 into master Dec 14, 2021
@marco-c marco-c deleted the remove-macro branch December 14, 2021 09:25
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.

4 participants