Fix unused types warnings

EcoJulia / Phylo.jl

Simple phylogenetic trees in Julia to work with Diversity.jl - https://github.com/EcoJulia/Diversity.jl

BSD 2-Clause "Simplified" License

33 stars 13 forks source link

Fix unused types warnings #82

Closed cossio closed 1 year ago

cossio commented 1 year ago

Attempt to fix #81

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5474312604

1 of 2 (50.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.8%) to 58.165%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/plot.jl	0	1	0.0%
<!--	Total:	1	2	50.0%	-->

Totals
Change from base Build 4814055898:	-1.8%
Covered Lines:	1268
Relevant Lines:	2180

💛 - Coveralls

richardreeve commented 1 year ago

Thanks. I think all of these changes are already in #79 as part of the general tidy up that @ejd99 has done, but that inference work is taking a bit of time, so happy to merge them now if you like.

cossio commented 1 year ago

Didn't realize #79 was covering this too. I think it's helpful to merge now just to get rid of the warning for now. And maybe tag a minor release with it