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.

Sign up for GitHub

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

Jump to bottom

Refactor code to use less dyn Trait #667

Open
hubcio opened this issue Feb 9, 2024 · 0 comments
Open

Refactor code to use less dyn Trait #667

hubcio opened this issue Feb 9, 2024 · 0 comments
Assignees
@haze518
Labels
good first issue Good for newcomers sdk Change related to sdk (client) API server iggy-servere related change

Comments

@hubcio
Copy link
Contributor

hubcio commented Feb 9, 2024

No description provided.

@hubcio hubcio added good first issue Good for newcomers server iggy-servere related change sdk Change related to sdk (client) API labels Feb 9, 2024
@haze518 haze518 self-assigned this Jan 13, 2025
@haze518 haze518 mentioned this issue Jan 16, 2025
Merged
spetz pushed a commit that referenced this issue Jan 20, 2025
@haze518 @hubcio
Refactor: Replace dyn Trait with Enum for Encryptor and Persister Typ…
5f2a435 
…es (#1442)

Addresses part of ##667

This PR refactors the codebase to replace the use of dyn Trait for
Encryptor and Persister with enum types (EncryptorKind and
PersisterKind)

Co-authored-by: Hubert Gruszecki <[email protected]>
numinnex pushed a commit that referenced this issue Jan 24, 2025
@haze518 @hubcio @numinnex
Refactor: Replace dyn Trait with Enum for Encryptor and Persister Typ…
b4aab01 
…es (#1442)

Addresses part of ##667

This PR refactors the codebase to replace the use of dyn Trait for
Encryptor and Persister with enum types (EncryptorKind and
PersisterKind)

Co-authored-by: Hubert Gruszecki <[email protected]>
@haze518 haze518 mentioned this issue Jan 25, 2025
Merged
spetz pushed a commit that referenced this issue Jan 26, 2025
@haze518
Refactor: Replace dyn Trait with Enums (#1462)
6b2695f 
Addresses part of #667

This PR refactors the codebase to replace the use of dyn Trait with
enums for Sender, Archiver, State, Storage, and ConfigProvider
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@haze518 haze518

Labels
good first issue Good for newcomers sdk Change related to sdk (client) API server iggy-servere related change
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hubcio @haze518