-
Notifications
You must be signed in to change notification settings - Fork 56
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
✨ New exporter: go #405
✨ New exporter: go #405
Conversation
9a386ec
to
0438348
Compare
Could you possibly add a README in https://github.com/COVESA/vss-tools/tree/master/docs that gives a bit on background and how the tool can be used or is intended to be used. Like linking to https://go.dev/ as possible not every one is familiar with Go. We use Go in https://github.com/COVESA/vissr and I am curious if what we have here could be a replacement to the "binary" format currently used by that tool. ideas @UlfBj ? |
Done. Added README: https://github.com/sschleemilch/vss-tools/blob/feature/go-exporter/docs/go.md |
450d3d4
to
bd026d6
Compare
Lets not merge it yet. I would like to make some improvements. |
MoM:
|
It looks good to me. |
5d11dcf
to
298f07c
Compare
Signed-off-by: Sebastian Schleemilch <[email protected]>
298f07c
to
fe7268e
Compare
MoM: Ok to merge |
Added new generator: go
It does produce structs to work with the vss tree.
It only produces simple structs for now. In future, we can also generate get/set methods and use the
allowed
as well asmin/max
fields.