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/add findex redis v7 #20

Open
wants to merge 56 commits into
base: develop
Choose a base branch
from
Open

Feat/add findex redis v7 #20

wants to merge 56 commits into from

Conversation

HatemMn
Copy link
Collaborator

@HatemMn HatemMn commented Dec 3, 2024

No description provided.

Manuthor and others added 30 commits October 18, 2024 10:25
Co-authored-by: Hatem M'naouer <[email protected]>
* ci: fix

* test: authentication failing

* test:  ignore auth tests

* test: re-enable test server auth tests

* test: re-enable client auth tests

* test: disable client auth tests

* test: do not clear db

* chore: rename access to permission

* refactor: check_permission

* test: try mt tests

* chore: remove all lints

* test: remove dependency_on_unit_never_type_fallback

* test: make default server port to 6666

* fix: lint
@HatemMn HatemMn changed the base branch from develop to reuse_cli December 3, 2024 13:06
@HatemMn HatemMn self-assigned this Dec 3, 2024
@HatemMn HatemMn force-pushed the feat/add_findex_redis_v7 branch from 52b6f6f to a5126b4 Compare December 3, 2024 16:22
@HatemMn HatemMn force-pushed the feat/add_findex_redis_v7 branch from a5126b4 to 9f5de93 Compare December 3, 2024 16:22
pub(crate) trait FindexTrait: Sync + Send {
use super::redis::WORD_LENGTH;

pub(crate) trait FindexMemoryTrait:
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

INFO : to use dyn capabalities we should assemble all required traits into one super trait

Base automatically changed from reuse_cli to findex_v6 December 5, 2024 12:09
Base automatically changed from findex_v6 to develop December 9, 2024 14:24
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