-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat: add option for generate
command to only generate allowed message ids
#248
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good, could you please add some tests and rebase on current master?
8a94a07
to
4a01aaf
Compare
Please also rebase/fixup commits so that each commit forms one logical and coherent change, incorporate any fix/revert commits to the original commits. Also we use Conventional Commits for our commit messages, please update your commits to conform :) |
9420312
to
912a0de
Compare
generate
command to only generate allowed message ids
generate
command to only generate allowed message idsgenerate
command to only generate allowed message ids
912a0de
to
95a09c7
Compare
95a09c7
to
d9ece70
Compare
This PR adds an message ID list option to only compile particular messages:
generate <input-dir> <output-dir> [<allowed-message-ids>...]
Example:
generate dbcIn dbcOut 587 256