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

Lock Smithy to a static version #327

Merged
merged 1 commit into from
Oct 24, 2024
Merged

Lock Smithy to a static version #327

merged 1 commit into from
Oct 24, 2024

Conversation

JordonPhillips
Copy link
Contributor

This updates our Smithy versioning to a single static version that is not synced via gradle properties. The version currently locked in is 1.52.0, which is not the latest, but 1.52.1 has a bug that prevents us from generating protocol tests.

Instead of relying on the version range to keep us up to date, we will now be relying on dependabot. It can also track all the different dependency file locations, which removes the need for the syncing via gradle properties.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This updates our Smithy versioning to a single static version that is
not synced via gradle properties. The version currently locked in is
1.52.0, which is not the latest, but 1.52.1 has a bug that prevents
us from generating protocol tests.

Instead of relying on the version range to keep us up to date, we
will now be relying on dependabot. It can also track all the
different dependency file locations, which removes the need for the
syncing via gradle properties.
@JordonPhillips JordonPhillips requested a review from a team as a code owner October 24, 2024 15:00
Copy link
Contributor

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

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

Is this something dependabot can manage for us going forward or are we going to need to come back to update all these places? I can read :)

@nateprewitt nateprewitt merged commit fda945a into develop Oct 24, 2024
5 checks passed
@nateprewitt nateprewitt deleted the lock-smithy-version branch October 24, 2024 15:05
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