feat: Refactor - Githubissues

jiro4989 / nimjson

nimjson generates nim object definitions from json documents.

https://jiro4989.github.io/nimjson/nimjson.html

MIT License

52 stars 5 forks source link

feat: Refactor #24

Closed jiro4989 closed 2 years ago

jiro4989 commented 2 years ago

変更内容

jsonの解析と一緒に文字列を組み立てるような処理がはっきり言って汚かったのをリファクタリングした
- 解析と文字列の生成は別の処理に分離
- 最初に解析だけ行って、Object定義Objectを生成する
- 次にObject定義Objectのプロシージャで文字列を生成する
合わせてモジュール構成を変更
- utilモジュールを削除
- typesモジュールを追加
- parserモジュールを追加
CIでのテストを追加
ユニットテストに testament を使うように変更
utilをinlucdeしていたのをやめてimportするように変更
- ただしインタフェースを破壊しないように、今までのプロシージャと同じインタフェースのものを残すようにした
docs の生成処理をプロジェクトビルドするように変更

破壊的変更 BREAKING CHANGES

nimjsonpkg/util.nim の削除
- 普通は nimjson をimportして使うことしか想定していないので、utilを直接参照してる人はいないはず・・・とは思いつつ、破壊的変更は破壊的変更なので
- nimjsonのtoTypeStringの挙動自体は変わっていない（はず）