openworm / behavioral_syntax

behavioral syntax analysis based on the paper of Andre Brown
Other
5 stars 1 forks source link

Solve build fail problems and coverage issue #18

Closed ghost closed 7 years ago

ghost commented 8 years ago

1) Build must pass 2) 100% coverage required.

MichaelCurrie commented 8 years ago

I believe https://github.com/AidanRocke/behavioral_syntax/pull/29 fixes your step number 1.

ghost commented 8 years ago

Just had it merged. I'm not yet sure how to do 100% test coverage or whether that's even necessary. But, I think it might be a good idea to have some measure of what percentage of the codebase has been tested.

It's definitely not high priority but here's more info: http://blog.jbrains.ca/permalink/on-100-unit-test-coverage-and-other-nonsensical-ideas

ghost commented 7 years ago

In retrospect this is a regular issue. Will close in order to avoid creating 'issue noise'.