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

Move tlmcmddb dir #172

Merged
merged 4 commits into from
Oct 30, 2023
Merged

Move tlmcmddb dir #172

merged 4 commits into from
Oct 30, 2023

Conversation

sksat
Copy link
Member

@sksat sksat commented Oct 26, 2023

概要

C2A user の tlmcmddb を置くディレクトリを <C2A user top>/tlm-cmd-db に変える

Issue

検証結果

コード生成 CI が通ればよし

影響範囲

  • C2A user の tlmcmddb を置くディレクトリ
  • code-generator の設定
  • C2A Boom の build:tlmcmddb の設定

@sksat sksat added priority::medium priority medium tools labels Oct 26, 2023
@sksat sksat self-assigned this Oct 26, 2023
@meltingrabbit
Copy link
Member

crateではないので tlm_cmd_db ?

@meltingrabbit
Copy link
Member

あと,cmdfileもあるので,tlm cmd db ではなさそう

@sksat
Copy link
Member Author

sksat commented Oct 27, 2023

code-generatorenum-loader もあるので,- 区切りは crate に限らないです.あと,これの場合は CSV だけでなくエディタである xlsm もあるので,実質的に https://github.com/ut-issl/tlm-cmd-db/ が入っているようなものでは?となったので名前を揃えてみました.cmdfile はそもそもこれら(cmd db, tlm db)と並列していないと思うので,さらに別のディレクトリに入れる方が適切かもしれないです.

@meltingrabbit
Copy link
Member

なるほど.いったん tlm-cmd-dbにcmdfileふくめて全部入れる,OKです.

ところで,レビューレディー?(レビュアー設定も,ディスクリプションの記載も途中っぽそうなので一応メンション) @sksat

@sksat
Copy link
Member Author

sksat commented Oct 30, 2023

@meltingrabbit review ready です

@meltingrabbit meltingrabbit self-requested a review October 30, 2023 04:33
@sksat sksat merged commit dcb1065 into develop Oct 30, 2023
35 checks passed
@sksat sksat deleted the feature/move-tlmcmddb-dir branch October 30, 2023 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

テレコマDBのパスを変えて標準化する
2 participants