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

Update v4 changelog #157

Merged
merged 8 commits into from
Nov 1, 2023
Merged

Update v4 changelog #157

merged 8 commits into from
Nov 1, 2023

Conversation

sksat
Copy link
Member

@sksat sksat commented Oct 17, 2023

概要

SSIA

Issue

@sksat sksat added the documentation Improvements or additions to documentation label Oct 17, 2023
@sksat sksat self-assigned this Oct 17, 2023
@sksat sksat added the priority::high priorityg high label Oct 27, 2023
@sksat
Copy link
Member Author

sksat commented Oct 27, 2023

rebase

@sksat sksat force-pushed the feature/update-v4-changelog branch 2 times, most recently from 829dc03 to 8f9d3f5 Compare October 30, 2023 04:49
@sksat sksat force-pushed the feature/update-v4-changelog branch from 698f0c1 to cd7b6c5 Compare October 30, 2023 08:42
@sksat sksat marked this pull request as ready for review October 30, 2023 08:52
@sksat
Copy link
Member Author

sksat commented Oct 30, 2023

@meltingrabbit まだ加筆修正するかもですが,大体書いたので確認お願いします

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
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 リポジトリに吸収した

Copy link
Member

Choose a reason for hiding this comment

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

この行があるせいで,mdの箇条書きのレンダリングがこわれてそう

Copy link
Member Author

Choose a reason for hiding this comment

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

壊れてなくないです?(どこのことを言ってる?)

Copy link
Member Author

Choose a reason for hiding this comment

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

あー,なんか行間がデカくなるやつのことか

Copy link
Member

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) の変更を取り込む
Copy link
Member

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 になってたはず)

Copy link
Member Author

Choose a reason for hiding this comment

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

混ざってないと思う

Copy link
Member

@meltingrabbit meltingrabbit Oct 31, 2023

Choose a reason for hiding this comment

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

ここでは混ざってないが,混ざると壊れるので,インデントは4 spaceに統一したい(してるので(まあ明文化してはないが))から,ここでも4 spaceでお願い,という趣旨だった

Copy link
Member Author

Choose a reason for hiding this comment

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

OK(この部分に対する指摘なのかと思った)

Copy link
Member Author

Choose a reason for hiding this comment

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

でもそもそも ol と ul を混ぜたい時がおかしくない?という気がするのであんまり納得いってないな.まあ趣味の問題だとは思うんですが.

Copy link
Member Author

Choose a reason for hiding this comment

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

そして別にインデントで挙動変わらなくないか

Copy link
Member

Choose a reason for hiding this comment

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

そもそも ol と ul を混ぜたい時がおかしくない?

混ざるのは正常では?
たとえば,これ(olは手順,ulはそれぞれの中での項目)

https://github.com/arkedge/c2a-core/blob/2761f809aeae3f8992e76e18e2ecaf566801c461/docs/general/release.md#%E6%9C%AC-release

そして別にインデントで挙動変わらなくないか

ここではulのみなので挙動は変わらないです.(混ざったときに壊れる(少なくとも当時は))

Copy link
Member Author

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 で見ただけだけど. > 挙動

Copy link
Member

Choose a reason for hiding this comment

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

あ,直ってるのか.
となるとどっちでもいいが,統一したさはある < 2 or 4 space

Copy link
Member

Choose a reason for hiding this comment

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

2spaceにそろえますか?

@sksat
Copy link
Member Author

sksat commented Nov 1, 2023

PR の一覧はマージ順の方がよい説などもあるけど,内容としてはこんなかんじでよいと思うので一旦これでマージ

@sksat sksat merged commit 054d0eb into develop Nov 1, 2023
35 checks passed
@sksat sksat deleted the feature/update-v4-changelog branch November 1, 2023 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority::high priorityg high
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants