Skip to content

rm getprop on cf.manifold#1793

Merged
Affie merged 1 commit intomasterfrom
23Q4/depr/cfmani
Oct 25, 2023
Merged

rm getprop on cf.manifold#1793
Affie merged 1 commit intomasterfrom
23Q4/depr/cfmani

Conversation

@dehann
Copy link
Copy Markdown
Member

@dehann dehann commented Oct 20, 2023

we're probably not going for a dep warning as this was never released

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 20, 2023

Codecov Report

Merging #1793 (b8aa3af) into master (c85fdb9) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1793      +/-   ##
==========================================
- Coverage   76.75%   76.72%   -0.04%     
==========================================
  Files          82       82              
  Lines        5954     5950       -4     
==========================================
- Hits         4570     4565       -5     
- Misses       1384     1385       +1     
Files Coverage Δ
src/entities/CalcFactor.jl 37.50% <ø> (-20.84%) ⬇️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Affie Affie marked this pull request as ready for review October 25, 2023 09:35
@Affie
Copy link
Copy Markdown
Member

Affie commented Oct 25, 2023

This looks good to me, we can merge this and then make sure it's not used downstream in RoME and Caesar.

@Affie Affie merged commit 4637ec6 into master Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants