Skip to content

Conversation

@ianbotsf
Copy link
Contributor

@ianbotsf ianbotsf commented Jul 1, 2025

Issue #

(none)

Description of changes

Removes @Deprecated APIs from the runtime modules in preparation for v1.5 release.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ianbotsf ianbotsf requested a review from a team as a code owner July 1, 2025 20:30
@ianbotsf ianbotsf added the acknowledge-api-break Acknowledge that a change is API breaking and may be backwards-incompatible. Review carefully! label Jul 1, 2025
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@ianbotsf ianbotsf added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Jul 1, 2025
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@@ -1,3 +1,4 @@

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: stray newline

public suspend fun parseEc2QueryErrorResponse(payload: ByteArray): ErrorDetails =
parseEc2QueryErrorResponseNoSuspend(payload)

public fun parseEc2QueryErrorResponseNoSuspend(payload: ByteArray): ErrorDetails {
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we have a FIXME to rename this back to parseEc2QueryErrorResponse in the future? Or maybe it's too much churn, not worth the effort?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No time like the present! I'll add that into this PR.

@github-actions

This comment has been minimized.

@ianbotsf ianbotsf merged commit 54dae6a into v1.5-main Jul 2, 2025
16 checks passed
@ianbotsf ianbotsf deleted the chore-remove-deprecated-apis branch July 2, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

acknowledge-api-break Acknowledge that a change is API breaking and may be backwards-incompatible. Review carefully! acknowledge-artifact-size-increase no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants