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

一部のレコードで「市区町村名ローマ字」フィールドが途切れている問題を修正 #123

Merged
merged 12 commits into from
Oct 4, 2022

Conversation

champierre
Copy link
Contributor

@champierre champierre commented Sep 29, 2022

APIの更新もコミットしてしまったため、内容が巨大になってしまっていますが、メインの変更は、patches 以下の変更を入れ込む処理を bin/build-api.js から bin/build.js に移動し、api だけでなく latest.csv および latest.db にも反映させたことです。

@champierre champierre changed the title Issue 118 2 patches 以下で追加された情報を latest.csv に追加する Sep 29, 2022
@champierre champierre force-pushed the issue_118_2 branch 5 times, most recently from 9fbef56 to 844f06a Compare September 30, 2022 03:38
Copy link
Contributor

@kamataryo kamataryo left a comment

Choose a reason for hiding this comment

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

ありがとうございます。
API に大量に差分が出ているのは JSO もソートされたからという理解でいいでしょうか?
そういうことでしたら OK だと思います!

@kamataryo
Copy link
Contributor

@champierre コンフリクトの解消だけお願いします 🙇

@kamataryo
Copy link
Contributor

あ、 patches/issue-118.json はもしかしてももう不要ですかね?

@champierre
Copy link
Contributor Author

ありがとうございます。 API に大量に差分が出ているのは JSO もソートされたからという理解でいいでしょうか? そういうことでしたら OK だと思います!

はい、そのとおりです。

@champierre
Copy link
Contributor Author

あ、 patches/issue-118.json はもしかしてももう不要ですかね?

issue-118.json は必要で、issue-118.csv が不要なので削除します。

@champierre
Copy link
Contributor Author

champierre commented Oct 4, 2022

latest.csv から抜き出してきた csv を patch 用の json 形式に変換するために使用した csv-to-json.js は gist に移したので、削除します。

https://gist.github.com/champierre/ddf80c3079eeade45a9b0f00f2ede75b

@kamataryo
Copy link
Contributor

ありがとうございます、マージします!

@kamataryo kamataryo merged commit 99bbc52 into develop Oct 4, 2022
@kamataryo kamataryo deleted the issue_118_2 branch October 4, 2022 03:10
@champierre champierre changed the title patches 以下で追加された情報を latest.csv に追加する 一部のレコードで「市区町村名ローマ字」フィールドが途切れている問題を修正 Oct 4, 2022
@champierre
Copy link
Contributor Author

Closes #118

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