search

datagouv / hydra

Async metadata crawler for data.gouv.fr
3 stars 0 forks source link

feat: add tests coverage #122

Closed bolinocroustibat closed 3 months ago

bolinocroustibat commented 3 months ago

Closes https://github.com/datagouv/hydra/issues/121.

bolinocroustibat commented 3 months ago

Test coverage, nice! Do you want to add this to the CHANGELOG?

Indeed, fixed

Some future discussion maybe: do we want to enable a CI target to fail when a PR lowers the global test coverage?

That's a good idea, but let's not make the stack overkill though, and slowing us down for fixes and features. This should only be done if this not becomes a blocker when needing to push fixes quickly. We can create a ticket as a reminder!