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.3をmainブランチにマージ #127

Merged
merged 37 commits into from
Jan 12, 2024

Conversation

YuukiToriyama
Copy link
Owner

No description provided.

YuukiToriyama and others added 30 commits January 7, 2024 21:10
都道府県マスタを取得するためのモジュール
市区町村マスタを取得するためのモジュール
`client.rs`, `blocking.rs`にそれぞれ書いていたものを移植している
`client.rs`, `blocking.rs`にそれぞれ書いていたものを移植している
…asterApiを持つような実装に変更

ライフタイムの関係でコンパイルができないため
…piとCityMasterApiを持つような実装に変更

ライフタイムの関係でコンパイルができないため
…ailed/re-organization-api-module

API失敗時のテストコード: `api`モジュールの再編
…ailed/add-test

API失敗時のテストコード: 都道府県マスタ/市区町村マスタが取得できない場合のテストコードを追加
…ailed/update-readme

API失敗時のテストコード: READMEのサンプルコードを修正
…nts-hinohara/hinohara-village

檜原の表記ゆれ: 「西多摩郡檜原村」と「西多摩郡桧原村」の表記ゆれに対応
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [clippy] reported by reviewdog 🐶

warning: you should consider adding a `Default` implementation for `Parser`
  --> src/lib.rs:18:5
   |
18 | /     pub fn new() -> Self {
19 | |         Parser {}
20 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
   = note: `#[warn(clippy::new_without_default)]` on by default
help: try adding this
   |
16 + impl Default for Parser {
17 +     fn default() -> Self {
18 +         Self::new()
19 +     }
20 + }
   |

@YuukiToriyama YuukiToriyama merged commit f202641 into main Jan 12, 2024
3 checks passed
@YuukiToriyama YuukiToriyama deleted the release/v0.1.0-beta.3 branch October 31, 2024 16:18
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.

「檜原村」「桧原村」の表記のゆれ API失敗時のテストコードを書く
1 participant