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

yqコマンドに-oyオプション付与 #3719

Merged
merged 2 commits into from
Oct 29, 2023
Merged

yqコマンドに-oyオプション付与 #3719

merged 2 commits into from
Oct 29, 2023

Conversation

massongit
Copy link
Member

@massongit massongit commented Oct 24, 2023

https://github.com/dev-hato/hato-atama/actions/runs/6618408201/job/18020431495?pr=3715

22:20:33 initCommand [WARN] JSON file output is now JSON by default (instead of yaml). Use '-oy' or '--output-format=yaml' for yaml output

上記WarningがElmのバージョン取得の箇所で出ている、かつ、YAML形式で出力すると -r オプションを付けなくてもダブルクオートなしの文字列になるので、該当箇所に -oy オプションを付与します。

@github-actions
Copy link
Contributor

9c3c89e のPR用環境: サイト, ログ

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Device URL
mobile https://v10566-dot-hato-atama.an.r.appspot.com

Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead

@massongit massongit changed the title yqでJSON形式を扱う際に-ojオプション付与 yqでJSON形式を扱う際に-oyオプション付与 Oct 24, 2023
@massongit massongit changed the title yqでJSON形式を扱う際に-oyオプション付与 yqコマンドに-oyオプション付与 Oct 24, 2023
@github-actions
Copy link
Contributor

1ce3f1e のPR用環境: サイト, ログ

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Device URL
mobile https://v10569-dot-hato-atama.an.r.appspot.com

Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead

@massongit massongit added this pull request to the merge queue Oct 26, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 26, 2023
@massongit massongit added this pull request to the merge queue Oct 29, 2023
@massongit massongit removed this pull request from the merge queue due to a manual request Oct 29, 2023
@massongit massongit added this pull request to the merge queue Oct 29, 2023
Merged via the queue into master with commit 3b661a9 Oct 29, 2023
52 checks passed
@massongit massongit deleted the add_oj branch October 29, 2023 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants