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

[code-generator] max_tlm_num のアサーションを追加 #243

Merged
merged 2 commits into from
Dec 18, 2023
Merged

[code-generator] max_tlm_num のアサーションを追加 #243

merged 2 commits into from
Dec 18, 2023

Conversation

meltingrabbit
Copy link
Member

@meltingrabbit meltingrabbit commented Dec 15, 2023 •
edited

概要

max_tlm_num を変更することが多くなってきたので，有効な tlm id の範囲を超えてないかを確認するようにした

備考

@meltingrabbit meltingrabbit added enhancement New feature or request tools labels Dec 15, 2023
@meltingrabbit meltingrabbit self-assigned this Dec 15, 2023
ToshiAki64
ToshiAki64 approved these changes Dec 18, 2023
View reviewed changes
Copy link
Collaborator

@ToshiAki64 ToshiAki64 left a comment

Choose a reason for hiding this comment

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

いいと思います。

Base automatically changed from feature/add_metadata_for_generator to main December 18, 2023 04:35
@meltingrabbit meltingrabbit merged commit b00e0e3 into main Dec 18, 2023
37 checks passed
@meltingrabbit meltingrabbit deleted the feature/add_asset_to_coge_gen branch December 18, 2023 04:40
@ToshiAki64 ToshiAki64 mentioned this pull request Jan 18, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ToshiAki64 ToshiAki64 ToshiAki64 approved these changes

@hender14 hender14 Awaiting requested review from hender14

Assignees

@meltingrabbit meltingrabbit

Labels
enhancement New feature or request tools
Projects
C2A public
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@meltingrabbit @ToshiAki64