deepset-ai / haystack-experimental

🧪 Experimental features for Haystack
https://haystack.deepset.ai
Apache License 2.0
23 stars 9 forks source link

ci: separate jobs for unit and integration tests #102

Closed anakin87 closed 1 month ago

anakin87 commented 1 month ago

Related Issues

Proposed Changes:

How did you test it?

CI

Notes for the reviewer

With this PR, integration tests depend on the linting job. We can make different choices, but this seemed reasonable to me to avoid long running times.

I also experimented with cache (with an approach similar to that used in Haystack). Still, I encountered some issues and I also found out that restoring the cache is slow in some environments (Windows), so I decided to avoid it for the time being.

Checklist

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 11034137945

Details


Totals Coverage Status
Change from base Build 11029532546: -3.7%
Covered Lines: 1429
Relevant Lines: 1583

💛 - Coveralls