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

BCE Rotator の実行開始コマンド位置の修正 #319

Merged
merged 4 commits into from
Mar 13, 2024

Conversation

ToshiAki64
Copy link
Collaborator

@ToshiAki64 ToshiAki64 commented Mar 10, 2024

概要

Rotator において,0 番目のコマンドから実行されることを意図していたが,実際は 1 番目のコマンドから実行されていたため, 0 番目から実行されるように修正

Issue

検証結果

examples のデバッグ出力の出方が変わることを確認
(その後、既存と同じ出力になるように examples 側を変更)

@ToshiAki64 ToshiAki64 self-assigned this Mar 10, 2024
@ToshiAki64
Copy link
Collaborator Author

@meltingrabbit
rotator の実行順序を修正しました。

@meltingrabbit
Copy link
Member

rotator の修正 の修正は良さそうなので、 migration guide も書いて頂けると助かります.

@ToshiAki64
Copy link
Collaborator Author

migration guide も書いて頂けると助かります

こちら、了解しました。
記載するのは、Release 時でよかったですか?

@meltingrabbit
Copy link
Member

記載するのは、Release 時でよかったですか?

このPRで書いておいてもらえると助かります.release時に私の方で他のPRなどまとめる予定です.
またなにかあれば追記・加筆をお願いするかもです.

@ToshiAki64
Copy link
Collaborator Author

起草してみましたので、確認いただけますか。

Copy link
Member

@meltingrabbit meltingrabbit left a comment

Choose a reason for hiding this comment

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

良さそうです.

CIに関わる別PRがマージされたので,最新のmainにrebaseして,念のためCIを通し直してからマージお願いします.

@ToshiAki64
Copy link
Collaborator Author

rebase して、CI 通ったので、マージします。

@ToshiAki64
Copy link
Collaborator Author

@meltingrabbit
マージしようと思ったら、権限ありませんでした。

@meltingrabbit
Copy link
Member

@ToshiAki64 今後のことを考えて,maintainer権限を付与しました.マージできるかご確認ください cc @sksat

@ToshiAki64
Copy link
Collaborator Author

ありがとうございます。
マージします。

@ToshiAki64 ToshiAki64 merged commit 5fd48d0 into main Mar 13, 2024
37 checks passed
@ToshiAki64 ToshiAki64 deleted the feature/fix_rotator branch March 13, 2024 08:03
@meltingrabbit meltingrabbit changed the title rotator の修正 BCE Rotator の実行開始コマンド位置の修正 May 13, 2024
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.

2 participants