Skip to content
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

Investigate non-atomic update to graal-sdk dependency in dependencies.properties file #2210

Open
mpeddada1 opened this issue Oct 30, 2023 · 2 comments
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@mpeddada1
Copy link
Contributor

With java-shared-config:1.6.0, the dependency management of graal-sdk in maven was shifted from sdk-platform-java to java-shared-config. However, for bazel, we still need to make an additional update to the graal-sdk version in sdk-platform-java.

TODO: Investigate if it is feasible to consolidate the updates to the graal-sdk.

@blakeli0 blakeli0 added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. priority: p2 Moderately-important priority. Fix may not be included in next release. labels Nov 3, 2023
@alicejli
Copy link
Contributor

@mpeddada1 I forget - is this still an open question?

@mpeddada1
Copy link
Contributor Author

@alicejli thanks for the note. Apologies for missing this earlier. Yes, this still remains to be an issue. While the dependency is atomically updated in Maven, a manual update in bazel is still needed. That being said, modifying the label to feature request since it is a proposed enhancement to how the update is done as opposed to a regression.

@mpeddada1 mpeddada1 added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. and removed type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Development

No branches or pull requests

3 participants