search

stoeffel / elm-verify-examples

BSD 3-Clause "New" or "Revised" License
167 stars 14 forks source link

no config #110

Closed stoeffel closed 8 months ago

stoeffel commented 8 months ago

closes #109

What changes?

gampleman commented 8 months ago

This is awesome!

What do you think about making all default for applications and exposed-modules, README.md for packages? Slightly more annoying to explain in the readme, but it's probably the simplest way to use the tool?

stoeffel commented 8 months ago

What do you think about making all default for applications and exposed-modules, README.md for packages? Slightly more annoying to explain in the readme, but it's probably the simplest way to use the tool?

I actually wonder about just removing exposed-modules. Packages usually don't have that many modules and running all instead of just exposed isn't an overhead. What do you think? @gampleman

gampleman commented 8 months ago

🤷 I kind of like 'exposed-modules', but I also like simplicity... and chances are it's not going to be that big of a difference in the vast majority of cases.

stoeffel commented 8 months ago

@knuton any feedback before I merge and release?

stoeffel commented 8 months ago

closes https://github.com/stoeffel/elm-verify-examples/issues/67