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

Develop #438

Closed
wants to merge 15 commits into from
Closed

Develop #438

wants to merge 15 commits into from

Conversation

InnocentBug
Copy link
Collaborator

Description

Changes

Known Issues

Notes

Checklist

  • My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
  • I have updated the documentation to reflect my changes.
  • My code changes have been verified by automated tests and pass all relevant test scenarios.

InnocentBug and others added 15 commits December 1, 2023 12:22
* first baby steps to refactor api into smaller pieces.
* capsule some of the request calls

* final touches

* make mypy happy

* add missing file

* implement the new paginator design

* adjust tests to the new design

* trunk catches

* remove wrong file

* small improvements

* make mypy happy

* adjust expected result for doctest

* further smaller fixes

* fix vocab test

* automate cancelling outdated workflow runs

* fix doctest

* fix test expectations

* remove tests that don't really make sense. I don't think we should process user supplied URLs too much.

* fix trunk check

* make API and data schema smarter. Delay init of DBschema to raise API connection error at right place.

* full circle and we still don't have the correct search results

* reenable tests

* stupid mistakes

* remove unmerged paths

* add missing page increase

* add good changes

* less error prone

* raw is better for debugging

* fix import error

* remove obsolete test
@InnocentBug InnocentBug self-assigned this Mar 11, 2024
Copy link

trunk-io bot commented Mar 11, 2024

Merging to develop in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants