Skip to content

Make funcspace independent - #226

Merged
calixteman merged 1 commit into
mozilla:masterfrom
SoftengPoliTo:make-funcspace-independent
Aug 5, 2020
Merged

Make funcspace independent#226
calixteman merged 1 commit into
mozilla:masterfrom
SoftengPoliTo:make-funcspace-independent

Conversation

@Luni-4

@Luni-4 Luni-4 commented Jun 26, 2020

Copy link
Copy Markdown
Collaborator

This PR makes the FuncSpace struct independent of a parser and should be merged after #223 and #225

Thanks in advance for your review! :)

@Luni-4
Luni-4 requested a review from calixteman June 26, 2020 08:33
@codecov-commenter

codecov-commenter commented Jun 26, 2020

Copy link
Copy Markdown

Codecov Report

Merging #226 into master will decrease coverage by 0.03%.
The diff coverage is 44.44%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
- Coverage   17.32%   17.28%   -0.04%     
==========================================
  Files          46       46              
  Lines        5918     5918              
  Branches      698      698              
==========================================
- Hits         1025     1023       -2     
- Misses       4490     4493       +3     
+ Partials      403      402       -1     
Impacted Files Coverage Δ
rust-code-analysis-cli/src/web/metrics.rs 36.00% <0.00%> (ø)
src/output/dump_metrics.rs 33.33% <0.00%> (ø)
src/spaces.rs 59.18% <57.14%> (ø)
src/metrics/loc.rs 45.00% <0.00%> (-1.67%) ⬇️

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 967304c...77838b6. Read the comment docs.

@calixteman

Copy link
Copy Markdown
Collaborator

Could you rebase please ?

@Luni-4

Luni-4 commented Aug 4, 2020

Copy link
Copy Markdown
Collaborator Author

We need to merge #224 first

@Luni-4
Luni-4 force-pushed the make-funcspace-independent branch from b6fa108 to 77838b6 Compare August 4, 2020 14:04
@Luni-4

Luni-4 commented Aug 4, 2020

Copy link
Copy Markdown
Collaborator Author

@calixteman

Now it is ready to be merged, sorry for the mess :)

@calixteman
calixteman merged commit 6642808 into mozilla:master Aug 5, 2020
@calixteman

Copy link
Copy Markdown
Collaborator

Thx

@Luni-4
Luni-4 deleted the make-funcspace-independent branch August 5, 2020 15:25
spacedragon pushed a commit to spacedragon/rust-code-analysis that referenced this pull request Feb 2, 2021
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.

3 participants