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 api def 2023 11 15 #88

Merged
merged 5 commits into from
Nov 15, 2023
Merged

Update api def 2023 11 15 #88

merged 5 commits into from
Nov 15, 2023

Conversation

bearmini
Copy link
Collaborator

API 定義ファイルを更新しました。

主にソラカメ関連のコマンドのほか、sims list-status-history コマンドと stats harvest export コマンドが追加されました。

多くの既存のコマンドの文言が修正されました。

Copy link
Contributor

@makoto-soracom makoto-soracom left a comment

Choose a reason for hiding this comment

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

LGTM!

気になるところはありますが、blocker にしなくていいと思います。

  • 「ソラカメ対応カメラデバイス」は「ソラカメ対応カメラ」で良さそうです。
  • overwrite で指定した文字列の表示方法がわかりませんでした。

別途コメントは残してあります。

generators/assets/cli/ja.yaml Outdated Show resolved Hide resolved
description: コマンド実行環境(プロファイル)の設定や表示を行います。
summary: コマンド実行環境 (プロファイル) の設定や表示を行います。
description: コマンド実行環境 (プロファイル) の設定や表示を行います。
overwrite: プロファイル情報がすでに存在していても上書きします。
Copy link
Contributor

Choose a reason for hiding this comment

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

今回追加された overwrite のメッセージを表示する方法がわかりませんでした。
もしよかったら表示方法を教えてくださいー。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

configure --help で出てきますー

Copy link
Contributor

Choose a reason for hiding this comment

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

おお、いた!なるほど。何かを上書きするのではなくて --overwrite (CLI 独自オプション) の説明なんですねー。了解。

bearmini and others added 2 commits November 15, 2023 15:00
Co-authored-by: makoto-soracom <87067458+makoto-soracom@users.noreply.github.com>
@bearmini bearmini merged commit 3058866 into master Nov 15, 2023
3 checks passed
@bearmini bearmini deleted the update-api-def-2023-11-15 branch November 15, 2023 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants