Skip to content

Chore: use a single package for all Feature structs - #3278

Merged
berndverst merged 3 commits into
dapr:mainfrom
ItalyPaleAle:feature-generic
Jan 8, 2024
Merged

Chore: use a single package for all Feature structs#3278
berndverst merged 3 commits into
dapr:mainfrom
ItalyPaleAle:feature-generic

Conversation

@ItalyPaleAle

Copy link
Copy Markdown
Contributor

Also replaces golang.org/x/exp/slices with slices from the Go 1.21 stdlib

small spin-off from #3046

Also replaces `golang.org/x/exp/slices` with `slices` from the Go 1.21 stdlib

> small spin-off from dapr#3046

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
@ItalyPaleAle
ItalyPaleAle requested review from a team as code owners December 18, 2023 18:19
@berndverst

Copy link
Copy Markdown
Member

Where is the tracking issue for this? Why are we doing this? I would like some explanation - no PRs without discussion.

@berndverst berndverst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Background needed

@ItalyPaleAle

Copy link
Copy Markdown
Contributor Author

Where is the tracking issue for this?

Just a refactoring to avoid code duplication. Instead of having a Features struct defined in each package, and the Contains method repeated, it's now implemented just once (and re-exported using generics)

@berndverst
berndverst merged commit c47907d into dapr:main Jan 8, 2024
@ItalyPaleAle
ItalyPaleAle deleted the feature-generic branch January 8, 2024 18:25
eunicecompra pushed a commit to eunicecompra/components-contrib that referenced this pull request Jan 9, 2024
Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Bernd Verst <github@bernd.dev>
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