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

Repository が2つの箇所に存在している #4

Open
culumn opened this issue Dec 25, 2018 · 1 comment
Open

Repository が2つの箇所に存在している #4

culumn opened this issue Dec 25, 2018 · 1 comment
Labels
question Further information is requested

Comments

@culumn
Copy link

culumn commented Dec 25, 2018

Why

entity パッケージに Repository が存在している。

type GitHubRepository interface {
GetContributions(ctx context.Context, userName string, from time.Time, to time.Time) ([]int, error)
}
type SlackRepository interface {
GetWorkingCronChannel(ctx context.Context) (string, error)
}

What

一般的に、 Repository はアダプターとして扱われるから、 adapter/repository の一つで良いと思う。意図的に entity パッケージに Repository を定義してる理由とかある?

@culumn culumn added the question Further information is requested label Dec 25, 2018
@natumn
Copy link
Contributor

natumn commented Dec 25, 2018

entity内にRepositoryを定義しているのは意図的にやってます。
repositoryを明示的にentityに依存させるという意味でentity内で定義してみました。
うーん、いらなかった気もしてくる。

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

No branches or pull requests

2 participants