search

eclipse / kuksa.val.feeders

kuksa.val.feeders
Apache License 2.0
8 stars 22 forks source link

Add DDS provider #61

Closed lukasmittag closed 1 year ago

lukasmittag commented 1 year ago

@AkhilTThomas

lukasmittag commented 1 year ago

tests are missing as of now. Will be added in another commit.

lukasmittag commented 1 year ago

Looks good, only some minors that we possibly could ignore.

Possibly we should also mention ddsprovider on top level README (https://github.com/boschglobal/kuksa.val.feeders/blob/feature/DDS_feeder/README.md)

I did start ddprovider natively as well as from docker and did test from dds2val directory and all succeeded.

@SebastianSchildt should we merge or would you like to see a config file?