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

release/v0.1.6をmainブランチにマージ #354

Merged
merged 31 commits into from
Aug 11, 2024
Merged

release/v0.1.6をmainブランチにマージ #354

merged 31 commits into from
Aug 11, 2024

Conversation

YuukiToriyama
Copy link
Owner

@YuukiToriyama
add: モジュール分割: repositoryクレートを追加
9377319
@YuukiToriyama
add: モジュール分割: domainクレートを作成
bbf30e3 
エンティティやエラーを表す構造体などを格納する
@YuukiToriyama
add: モジュール分割: serviceクレートを作成
6776f0d
@YuukiToriyama
add: モジュール分割: repositoryクレートにgeoloniaモジュールを作成
100accf
@YuukiToriyama
update: モジュール分割: coreクレートにrepositoryクレートへの依存を追加
8a10b1b
@YuukiToriyama
update: モジュール分割: coreクレートにdomainクレートへの依存を追加
9e95a52
@YuukiToriyama
update: モジュール分割: coreクレートのerr.rsをdomainクレートのgeoloniaモジュールに移動
d3c4a4e
@YuukiToriyama
update: モジュール分割: core/src/entity.rsのParseResultを`core/src/parser.…
edda33f 
…rs`に移動
@YuukiToriyama
update: モジュール分割: テスト用のデータを生成するメソッドをsrc/entity.rsに移動
39f395c
@YuukiToriyama
update: モジュール分割: coreクレートのentity.rsをdomainモジュールのgeoloniaモジュールに移動
1891d86
@YuukiToriyama
add: モジュール分割: serviceクレートにGeoloniaApiServiceを作成した
339e719
@YuukiToriyama
update: モジュール分割: GeoloniaApiServiceに対してテストコードを作成
d54c5b8
@YuukiToriyama
update: モジュール分割: coreモジュールのCityMasterApiの実装をGeoloniaApiServiceベ…
8def6ff 
…ースになるよう修正
@YuukiToriyama
update: モジュール分割: coreモジュールのPrefectureMasterApiの実装を`GeoloniaApiSer…
4ad2c26 
…vice`ベースになるよう修正
@YuukiToriyama
update: モジュール分割: core/src/api/prefecture_master_api.rsをrepository…
5cbb4af 
…クレートの`geolonia`モジュールに移動
@YuukiToriyama
update: モジュール分割: core/src/api/city_master_api.rsをrepositoryクレートの`…
4ea9220 
…geolonia`モジュールに移動
@YuukiToriyama
update: モジュール分割: repositoryクレートにblockingフィーチャを追加
f8ba7e4
@YuukiToriyama
update: モジュール分割: serviceクレートにblockingフィーチャを追加
0558c1f
@YuukiToriyama
update: モジュール分割: coreからserviceへの依存を削除
f1de0d3 
`service`に直接依存するのではなく`repository`に依存する方が良いため
@YuukiToriyama
update: モジュール分割: serviceクレートのreqwest::blocking::*を使用しているメソッドは`blo…
f5371e1 
…cking`フィーチャを指定している時のみビルドするように設定

wasm-pack buildが失敗するため
@YuukiToriyama
update: モジュール分割: coreモジュールからreqwestの依存を削除
2ed035f 
ネットワークアクセスは`service`クレートの責務であり、`core`では使用されなくなったため
@YuukiToriyama
update: モジュール分割: 後方互換性を保つためにdomain::geolonia::entity::*および`core::pa…
b697846 
…rser::ParseResult`を`core/src/entity.rs`で再公開する

ただし、将来的には削除したいのでDeprecated指定する。
@YuukiToriyama
Merge pull request #349 from YuukiToriyama/feature/multi-module/domain
ef5a45a 
モジュール分割: `domain`モジュールを作成
@YuukiToriyama
Merge pull request #350 from YuukiToriyama/feature/multi-module/service
82a988b 
モジュール分割: `service`クレートを作成
@YuukiToriyama
Merge pull request #351 from YuukiToriyama/feature/multi-module/repos…
09ee817 
…itory

モジュール分割: `repository`クレートを作成
@YuukiToriyama
update: モジュール分割: CIの整備
76e6195 
新しく定義したクレートに対してもテストが走るように設定を修正。
併せて、テストの順序の変更や、ワークフローの分割などを行なった。

c.f. edb5518 eed7610 258601a
@YuukiToriyama
update: モジュール分割: READMEに貼っているCIのステータスバッチを更新
ba3e48d 
ワークフローが変更になるため
@YuukiToriyama
fix: モジュール分割: repositoryクレートにtokioの依存関係を追加
b620f4a 
非同期テストの実行に必要なため
@YuukiToriyama
Merge pull request #352 from YuukiToriyama/feature/multi-module/ci
63ad175 
モジュール分割: CIの整備
@YuukiToriyama
update-version: 0.1.5 -> 0.1.6
0871472
@YuukiToriyama
Merge pull request #353 from YuukiToriyama/feature/multi-module/master
3d7fbf6 
モジュール分割をrelease/v0.1.6にマージ
@YuukiToriyama YuukiToriyama added this to the モジュール分割 milestone Jul 28, 2024
@YuukiToriyama YuukiToriyama self-assigned this Jul 28, 2024
@YuukiToriyama YuukiToriyama merged commit f301d84 into main Aug 11, 2024
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@YuukiToriyama YuukiToriyama

Labels
None yet
Projects
None yet
Milestone
モジュール分割
Development

Successfully merging this pull request may close these issues.
1 participant
@YuukiToriyama