Skip to content

Updating dependencies to fix vulnerabilities#151

Merged
erikbosch merged 1 commit into
eclipse-velocitas:mainfrom
SoftwareDefinedVehicle:erik_dep
Oct 4, 2024
Merged

Updating dependencies to fix vulnerabilities#151
erikbosch merged 1 commit into
eclipse-velocitas:mainfrom
SoftwareDefinedVehicle:erik_dep

Conversation

@erikbosch
Copy link
Copy Markdown
Contributor

@erikbosch erikbosch commented Oct 2, 2024

There are some hypothetical vulnerabilities in v0.14.1 of this repo, so the examples and skeleton should preferably not use that one. I have just updated it to last release version (examples are anyway partially broken, see #141 )

We should better discuss how to handle these references when we do releases. Should we always update them before/after we create a new pypi release? Or should we in these two cases just remove version so we use "latest and greatest" velocitas-sdk

@erikbosch erikbosch marked this pull request as ready for review October 2, 2024 08:21
@erikbosch
Copy link
Copy Markdown
Contributor Author

@BjoernAtBosch - this was the starting point for the wiki definition - that I realized that we somewhere should document when to update these two version labels.

(As of today I think we can update them before we make a release as we do not test them in CI, but that might be more cumbersome if we actually starts testing them in CI as the new version do not yet exist in pypi when CI is run. I guess the examples better fit in another repo, especially as some of them like https://github.com/eclipse-velocitas/vehicle-app-python-sdk/tree/main/examples/seat-adjuster explicitly says that they cnanot be used from the sdk)

Copy link
Copy Markdown
Member

@BjoernAtBosch BjoernAtBosch left a comment

Choose a reason for hiding this comment

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

LGTM

@erikbosch erikbosch merged commit cf7e2ff into eclipse-velocitas:main Oct 4, 2024
@erikbosch erikbosch deleted the erik_dep branch October 4, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants