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

Document / prefer use of BOM for resolving SDK artifacts #1445

Open
cloudshiftchris opened this issue Oct 17, 2024 · 1 comment
Open

Document / prefer use of BOM for resolving SDK artifacts #1445

cloudshiftchris opened this issue Oct 17, 2024 · 1 comment
Labels
documentation This is a problem with documentation. p2 This is a standard priority issue

Comments

@cloudshiftchris
Copy link

Describe the issue

Currently the documentation here does not show or mention the BOM. Using the BOM should be the default / recommended approach, as it encapsulates the version constraints.

For example, using the BOM will prevent users from specifying different versions - perhaps accidentally or incidentally - for different service client dependencies (and perhaps other runtime dependencies such as http clients). This can happen even with the version catalog, as those versions are merely an input into dependency resolution (not a constraint) - other components (perhaps transitive dependencies) could specify a higher version of a specific JAR, leading to misalignment and subsequent quirky runtime behaviour.

The AWS Java SDK documents using the BOM here (also in other pages for Maven etc).

Links

https://docs.aws.amazon.com/sdk-for-kotlin/latest/developer-guide/setup-create-project-file.html

@cloudshiftchris cloudshiftchris added documentation This is a problem with documentation. needs-triage This issue or PR still needs to be triaged. labels Oct 17, 2024
@0marperez 0marperez added p2 This is a standard priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Oct 18, 2024
@0marperez
Copy link
Contributor

0marperez commented Oct 18, 2024

Hi, thanks for the report. I understand how this would be a good addition to our documentation. I've set this issue as standard priority (p2) and created an internal ticket for us to track this and implement into our sprints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

2 participants