-
Notifications
You must be signed in to change notification settings - Fork 22
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 Pull Request for EVA signature #172
Conversation
ECA works, YAAAY |
@@ -17,7 +17,6 @@ Name | Description | |||
* [SOME/IP Provider](https://github.com/eclipse-kuksa/kuksa-someip-provider) | |||
* [DDS Provider](https://github.com/eclipse-kuksa/kuksa-dds-provider) | |||
* [CSV Provider](https://github.com/eclipse-kuksa/kuksa-csv-provider) | |||
* [eCAL Provider](https://github.com/eclipse-kuksa/kuksa-incubation/tree/main/ecal2val) |
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.
I assume this is a merge conflict error, i.e. that it is not intentional to remove this line?
@@ -7,7 +7,7 @@ | |||
Name | Description | |||
---- | ----------- | |||
[Replay](./replay) |[KUKSA Server](https://github.com/eclipse/kuksa.val/tree/master/kuksa-val-server) replay script for previously recorded files, created by running KUKSA Server with `--record` argument **Note: Replay is deprecated and will reach End-of-Life 2024-12-31!** | |||
[ecal2val](./ecal2val) |Provider for eCAL data to [KUKSA Databroker](https://github.com/eclipse/kuksa.val/tree/master/kuksa_databroker) | |||
[fone2val](./fone2val) | F1 Telemetrydata source for [KUKSA Databroker](https://github.com/eclipse/kuksa.val/tree/master/kuksa_databroker) |
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.
Here removing the ecal2val line is fine as I better should have removed it in last commit
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 good to me.
There is some minor things which I believe is merge conflict regressions in main README but I do not see that as necessary as fix before merge as I anyway will need to update the README in the planned move of the content of this PR after merge to https://github.com/eclipse-kuksa/kuksa-incubation (as we are about to archive this repo)
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.
Then let's merge here. And "prepare for move" :)
Now I really need a PS...
Previously in https://github.com/eclipse/kuksa.val.feeders/tree/main/fone2val Original PR: eclipse/kuksa.val.feeders#172
I now have checked the email, unset and set it again and i hope everything is set up properly now.