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.

Sign up for GitHub

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

Jump to bottom

テレコマDBの階層を上に持ってくる #141

Merged
merged 1 commit into from
Jul 26, 2023
Merged

テレコマDBの階層を上に持ってくる #141

merged 1 commit into from
Jul 26, 2023

Conversation

chutaro
Copy link
Contributor

@chutaro chutaro commented Jul 25, 2023 •
edited
Loading

Issue

詳細

  • テレコマDBの階層が不自然に深いので、それを root 直下に持ってくる（C2A標準でそのように変更しようとしている）
  • 合わせて readme を微修正

検証結果

ビルドチェック (どちらもチェック)

  • SILSでのビルドチェックに通った(CIで確認)
  • vMicroでのビルドチェックに通った

動作確認チェック (いずれかをチェック)

  • SILSでアルゴリズムが想定通りに動いた
  • 実機でアルゴリズムが想定通りに動いた
  • (テレコマ試験の場合)コマンドファイルを使った試験をパスした

試験結果詳細記述場所 or 詳細ログ保存場所へのリンク

  • 図や表で記述する

補足

何かあれば書く。なければNAとする。

@chutaro chutaro requested review from a team and sksat as code owners July 25, 2023 07:01
@chutaro chutaro requested review from 200km, seki-hiro, suzuki-toshihir0 and conjikidow and removed request for a team July 25, 2023 07:01
@chutaro chutaro changed the title テレコマDBno テレコマDBの階層を上に持ってくる Jul 25, 2023
@chutaro chutaro self-assigned this Jul 25, 2023
@chutaro chutaro added the ✈️ priority::medium priority medium label Jul 25, 2023
@chutaro
Copy link
Contributor Author

chutaro commented Jul 25, 2023

CI 落ちているのは既知の不具合でしょうか？（このPRではソースコード変えていないので）

@200km
Copy link
Member

200km commented Jul 25, 2023

CIについては下記の通りです。

#128 (comment)

@200km 200km added the 🐳 major update Major update label Jul 25, 2023
@200km
Copy link
Member

200km commented Jul 25, 2023

major update相当だと思うので、マージは下記のやつを先にしたい

#140

200km
200km approved these changes Jul 25, 2023
View reviewed changes
Copy link
Member

@200km 200km left a comment

Choose a reason for hiding this comment

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

修正ありがとうございます。

@200km
Copy link
Member

200km commented Jul 25, 2023

@seki-hiro STTの修正とコンフリクトすると思うので、確認しておいてください。

@chutaro
Copy link
Contributor Author

chutaro commented Jul 26, 2023 •
edited
Loading

CIについては下記の通りです。

承知しました。

STTの修正とコンフリクトすると思う

@seki-hiro 上記すぐマージできそうだったら待ちます。まだ時間かかるようだったら、先にこのPRをマージしてから上記PRのコンフリクト解消するのは僕の方でやります。どうしますか？

@seki-hiro
Copy link
Member

時間かかるので先こっちマージで大丈夫です。

コンフリクト解消どうもありがとう。localでDBに少し変更加えてしまってるので、push次第上記PRでメンションします。

@chutaro chutaro merged commit 19a9760 into develop Jul 26, 2023
@chutaro chutaro deleted the feature/change_tlm_cmd_db_dir branch July 26, 2023 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@200km 200km 200km approved these changes

@sksat sksat Awaiting requested review from sksat sksat is a code owner

@seki-hiro seki-hiro Awaiting requested review from seki-hiro seki-hiro is a code owner automatically assigned from ut-issl/aocs

@suzuki-toshihir0 suzuki-toshihir0 Awaiting requested review from suzuki-toshihir0 suzuki-toshihir0 is a code owner automatically assigned from ut-issl/aocs

@conjikidow conjikidow Awaiting requested review from conjikidow conjikidow is a code owner automatically assigned from ut-issl/aocs

Assignees

@chutaro chutaro

Labels
✈️ priority::medium priority medium 🐳 major update Major update
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chutaro @200km @seki-hiro