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

アプリケーション追加時に必要な手順をドキュメントに追加する #299

Merged
merged 8 commits into from
Feb 1, 2024

Conversation

hender14
Copy link
Contributor

@hender14 hender14 commented Jan 23, 2024

Issue

詳細

c2a coreのドキュメントとして、
アプリケーション追加時に必要な手順を追加する。
また、合わせてコマンドの追加方法についても記載しておく

補足

ドキュメントの追加のみで、コードに変更はありません。

@hender14
Copy link
Contributor Author

@ToshiAki64
c2a coreのdocにアプリ追加方法に関するドキュメントを追加しました。
此方は秋間さんにreview頂く形で問題無いですか?

@ToshiAki64
Copy link
Collaborator

@meltingrabbit
こちら、鈴本さんにも確認いただきたいです。

@meltingrabbit
Copy link
Member

見ます.ちょいっと出張中なので,後回しになりますが...(ごめんなさい.)

@hender14
Copy link
Contributor Author

見ます.ちょいっと出張中なので,後回しになりますが...(ごめんなさい.)

急ぎではないので、後回しで問題ありません。

docs/README.md Outdated Show resolved Hide resolved
@hender14
Copy link
Contributor Author

@meltingrabbit
修正完了したので、再reviewお願い致します。

@meltingrabbit
Copy link
Member

@hender14 v4.2.0 でたので rebase お願いします

@meltingrabbit
Copy link
Member

@hender14 C2Aらしい言葉遣いに直してみました.確認いただけますか?

@hender14
Copy link
Contributor Author

hender14 commented Feb 1, 2024

C2Aらしい言葉遣いに直してみました.確認いただけますか?

修正ありがとうございます。
問題無さそうです。

@hender14
Copy link
Contributor Author

hender14 commented Feb 1, 2024

v4.2.0 でたので rebase お願いします

@meltingrabbit
rebase実施したので、再度reviewお願いします。

@hender14
Copy link
Contributor Author

hender14 commented Feb 1, 2024

@sksat
此方の画面だとMerging is blockedになってるんですが、
c2a-coreにmergeする場合リポジトリ側で何か設定が必要なんでしたっけ?

image

@meltingrabbit
Copy link
Member

@hender14 あぁ,これOSSなので,マージできる人制限していました.こちらでマージしますね.修正ありがとうございました!(おいおいマージ権限付与しますね)

@meltingrabbit meltingrabbit merged commit 284671d into main Feb 1, 2024
37 checks passed
@meltingrabbit meltingrabbit deleted the feature/doc_add_app branch February 1, 2024 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants