-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update v4 changelog #157
Update v4 changelog #157
Conversation
rebase |
829dc03
to
8f9d3f5
Compare
698f0c1
to
cd7b6c5
Compare
@meltingrabbit まだ加筆修正するかもですが,大体書いたので確認お願いします |
CHANGELOG.md
Outdated
|
||
### Compatible C2A Tools | ||
|
||
- [c2a-code-generator](./code-generator): [ut-issl/c2a-tlm-cmd-code-generator ae-v2.0.0](https://github.com/ut-issl/c2a-tlm-cmd-code-generator/releases/tag/ae-v2.0.0) を c2a-core リポジトリに吸収し,rename した | ||
- [c2a-enum-loader](./enum-loader): [ut-issl/c2a-enum-loader ae-v2.0.0](https://github.com/ut-issl/c2a-enum-loader/releases/tag/ae-v2.0.0) を c2a-core リポジトリに吸収した | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この行があるせいで,mdの箇条書きのレンダリングがこわれてそう
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
壊れてなくないです?(どこのことを言ってる?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あー,なんか行間がデカくなるやつのことか
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
それ
注意: migration 方法やスクリプトの修正などもありえるため,本 Migration Guide 及び Migration Script は c2a-core v4 系のその時点での最新版を参照すること | ||
|
||
- v4.0.0-alpha.0 | ||
- [#19](https://github.com/arkedge/c2a-core/pull/18) の変更を取り込む |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
githubのmdのレンダリング,olとulがまざると,インデントが4spaceじゃないと壊れるので,それで揃えてほしい(c2a docsは全部 4 space になってたはず)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
混ざってないと思う
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここでは混ざってないが,混ざると壊れるので,インデントは4 spaceに統一したい(してるので(まあ明文化してはないが))から,ここでも4 spaceでお願い,という趣旨だった
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK(この部分に対する指摘なのかと思った)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
でもそもそも ol と ul を混ぜたい時がおかしくない?という気がするのであんまり納得いってないな.まあ趣味の問題だとは思うんですが.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
そして別にインデントで挙動変わらなくないか
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
そもそも ol と ul を混ぜたい時がおかしくない?
混ざるのは正常では?
たとえば,これ(olは手順,ulはそれぞれの中での項目)
そして別にインデントで挙動変わらなくないか
ここではulのみなので挙動は変わらないです.(混ざったときに壊れる(少なくとも当時は))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
まあそうか......個人的には ol で複数段にならないようにするか,全部 ul に揃えるかにするけども...... > 混ざる
いや,ここの話ではなく,今は壊れないのでは,という話です.Issue comment の preview で見ただけだけど. > 挙動
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あ,直ってるのか.
となるとどっちでもいいが,統一したさはある < 2 or 4 space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2spaceにそろえますか?
PR の一覧はマージ順の方がよい説などもあるけど,内容としてはこんなかんじでよいと思うので一旦これでマージ |
概要
SSIA
Issue