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

マルチモジュール化をrelease/v0.1.0-beta.10にマージ #196

Merged
merged 45 commits into from
Mar 22, 2024

Conversation

YuukiToriyama
Copy link
Owner

@YuukiToriyama YuukiToriyama commented Mar 22, 2024

変更点

  • プロジェクトにcargo workspaceを導入した
  • これまでは1つのパッケージとして開発していたが、コードが増大化してきたためパッケージを意味ある区切りで分割していくことにした。
  • WebAssemblyのみならず複数のプラットフォームへの対応を進めていく上で、コアモジュールとそれぞれのプラットフォーム向けのコードを分離したい意図がある。

その他

YuukiToriyama and others added 30 commits February 25, 2024 23:38
…rs`の値をルートのCargo.tomlで管理するように変更
…re-module

マルチモジュール化: `core`モジュールの作成
…`, `categories`もルートのCargo.tomlで管理するように変更
`core`モジュールと`wasm`モジュールで使用するバージョンを揃えたいため
…を削除し、処理をベタがきにした

`core/lib.rs`から`Parser`を削除したいため
…move-usages-of-parser

マルチモジュール化: `Parser`の使用箇所を削除
…rge/main_f145995

マルチモジュール化にmainブランチを取り込み
`wasm-pack build`時にワーキングディレクトリにLICENSEがないとエラーが出るため
`wasm`モジュールを作成したことによるワークフローの修正
`wasm`モジュールを作成したことによるワークフローの修正
…x-workflows

マルチモジュール化: `core`モジュールを作成したことに伴うGitHubActionsワークフローの修正
YuukiToriyama and others added 14 commits March 6, 2024 16:17
…re-include-license-and-readme

マルチモジュール化: `core`にLICENSEを追加、Cargo.tomlにルートのREADMEを参照させる設定追加
- jobのworking-directoryを`core`から`wasm`に変更
- `wasm-pack build`によって生成されるpackage.jsonの`name`が`@toriyama/wasm`となっておりnpmjs.comに公開しているパッケージ名と異なってしまうため、jqコマンドを使用しpackage.jsonを修正するステップを追加
- クレート名が変わっていることにより、`wasm-pack build`で生成されるjsファイルの名前が変わっていたため、`--out-name japanese_address_parser`を指定するように変更
…futures`を削除

もはや必要ではなくなったため
…sm-module

マルチモジュール化: `wasm`モジュールを作成
…rge/main_005bbc2

マルチモジュール化にmainブランチを取り込み
@YuukiToriyama YuukiToriyama marked this pull request as ready for review March 22, 2024 07:33
@YuukiToriyama YuukiToriyama self-assigned this Mar 22, 2024
@YuukiToriyama YuukiToriyama merged commit e1a0ccd into release/v0.1.0-beta.10 Mar 22, 2024
2 checks passed
@YuukiToriyama YuukiToriyama deleted the feature/cargo-workspace/master branch December 29, 2024 12:56
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.

1 participant