search

naomijub / edn-rs

[DEPRECATED]: Crate to parse and emit EDN
https://crates.io/crates/edn-rs
MIT License
81 stars 14 forks source link

Code coverage failing after repo transfer #128

Closed Grinkers closed 9 months ago

Grinkers commented 9 months ago

Maybe we should just remove it? I don't think tarpaulin is particularly accurate anyway.

https://blog.rng0.io/how-to-do-code-coverage-in-rust/ Source-based code coverage seems to be the new hotness, but I think it has problems with merging features? I want to take a serious attempt at no_std. So I think all the coverage tooling will probably be a bit iffy. It already is with feature gating and macros.

I'm not really finding any good examples of any open source projects using any coverage tools with CI. I think maybe nothing is better than inaccurate/misleading.

Grinkers commented 9 months ago

Also, while doing cleanup or maintenance, maybe we rename the default branch to main?

Grinkers commented 9 months ago

Oh, sorry. It looks like it was a thing from me working on a fork. The other things mentioned are still valid, but this isn't a real issue.

naomijub commented 9 months ago

I can rename the branch to main in another issue

naomijub commented 9 months ago

Also, while doing cleanup or maintenance, maybe we rename the default branch to main?

Done @Grinkers @evaporei