Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: quivr core tox test + parsers #2929

Merged
merged 9 commits into from
Jul 30, 2024
Merged

Conversation

AmineDiro
Copy link
Collaborator

Description

  • Moved parser from quivr-api to quivr-core
  • Added tox tests with separate environements
  • Added megaparse dependency

Running the test

Running test scripts: "./scripts/run_tests.sh":

  • Build docker test image
  • Run tox with the configured env flags to run tests based on their markers and/or directory.

TODO:

  • For now the tika tests doesn't work @StanGirard : can we do a docker in docker for spawning a tika server 🤔
  • The tests depend

Copy link

vercel bot commented Jul 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
quivrapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 30, 2024 4:24pm

@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. area: backend Related to backend functionality or under the /backend directory labels Jul 30, 2024
@StanGirard
Copy link
Collaborator

@AmineDiro We should instead create our own docker image with tox and use this one to run the test in the CI

@StanGirard StanGirard merged commit 6855585 into main Jul 30, 2024
5 of 7 checks passed
@StanGirard StanGirard deleted the feat/quivr-core-tox-test branch July 30, 2024 16:49
AmineDiro pushed a commit that referenced this pull request Aug 2, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.0.13](core-0.0.12...core-0.0.13)
(2024-08-01)


### Features

* quivr core tox test + parsers
([#2929](#2929))
([6855585](6855585))


### Bug Fixes

* processor quivr version
([#2934](#2934))
([2d64962](2d64962))
* quivr core fix tests
([#2935](#2935))
([d9c1f3a](d9c1f3a))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
StanGirard added a commit that referenced this pull request Aug 7, 2024
🤖 I have created a release *beep* *boop*
---


## 0.0.294 (2024-08-07)

## What's Changed
* Delete Porter Application quivr-com by @porter-deployment-app in
#2927
* Delete Porter Application quivr-com-backend by @porter-deployment-app
in #2928
* feat: quivr core tox test + parsers by @AmineDiro in
#2929
* feat(frontend): handle no brain selection by @Zewed in
#2932
* fix: processor quivr version by @AmineDiro in
#2934
* fix: quivr core fix tests by @AmineDiro in
#2935
* chore(main): release core 0.0.13 by @StanGirard in
#2930
* feat: Add GitHub sync functionality to sync router by @chloedia in
#2871
* refactor: Remove syncGitHub function from useSync.ts by @StanGirard in
#2942
* feat: add chat with models by @StanGirard in
#2933
* ci: precommit in CI by @AmineDiro in
#2946
* feat: Add get_model method to ModelRepository by @StanGirard in
#2949
* feat: Add user email to StripePricingOrManageButton and
UpgradeToPlusButton components by @StanGirard in
#2951


**Full Changelog**:
v0.0.293...v0.0.294

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: backend Related to backend functionality or under the /backend directory size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants