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

Rename tools dir to Boom tools #332

Merged
merged 2 commits into from
May 16, 2024
Merged

Conversation

sksat
Copy link
Member

@sksat sksat commented Apr 11, 2024

概要

C2A user の tools ディレクトリを boom-tools に rename する

詳細

tools#56 で導入された C2A Boom のためのものであるため、名前で C2A Boom 関係のものだと明示するようにする

検証結果

example user のディレクトリで npm install して C2A DevTools などが使えればよし

影響範囲

C2A Boom 環境

@sksat sksat added priority::high priorityg high tools labels Apr 11, 2024
@sksat sksat requested review from KOBA789 and meltingrabbit April 11, 2024 05:44
@sksat sksat self-assigned this Apr 11, 2024
@sksat sksat modified the milestones: v5.0.0, v4.4.0 Apr 11, 2024
@meltingrabbit
Copy link
Member

「C2A Boom」という単語がどこにも定義されてないので,簡易でもいいのでドキュメント書いてもらえないですか?

cf. #49

@sksat
Copy link
Member Author

sksat commented Apr 11, 2024

書くのはいいんですがこの PR のスコープ外です

@KOBA789
Copy link
Member

KOBA789 commented Apr 11, 2024

大賛成

@meltingrabbit
Copy link
Member

meltingrabbit commented Apr 11, 2024

はい.PR分けてもらってもいいのですが,v4.4.0 にいれるかどうか気になってます.
v4.4.0 はだいぶ遅延しているので,ドキュメント PR が当面先なのであれば,この PR は v4.4.0 に含める必要はないかと思ってます.(v4.4.0 リリースを急ぎ先にやりたい)

@sksat
Copy link
Member Author

sksat commented Apr 11, 2024

この PR は現状の是正なので詳細を紹介するドキュメントが欲しい、という話は無関係です。C2A Boom という語自体は既に c2a-core の PR でも各 C2A user での実際の開発でも内部ドキュメントでも CHANGELOG にもあります。

@sksat
Copy link
Member Author

sksat commented Apr 11, 2024

ドキュメントの話とこの PR の話は別だが、ややユーザに驚きを与える可能性があるのとスケジュール的な観点から v4.4.0 の後にする

@sksat sksat removed this from the v4.4.0 milestone Apr 11, 2024
@sksat sksat added this to the v4.5.0 milestone May 16, 2024
@sksat
Copy link
Member Author

sksat commented May 16, 2024

rebase

@sksat sksat force-pushed the feature/rename-tools-dir-to-boom-tools branch from d667f1f to 72f7f7c Compare May 16, 2024 07:27
@sksat sksat changed the title Rename tools dir to boom tools Rename tools dir to Boom tools May 16, 2024
@sksat sksat merged commit dd9ed4d into main May 16, 2024
37 checks passed
@sksat sksat deleted the feature/rename-tools-dir-to-boom-tools branch May 16, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants