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.

Sign up for GitHub

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

Jump to bottom

kn: enable many @IgnoreNative tests #1233

Open
wants to merge 64 commits into
base: kn-main
Choose a base branch
from
Open

kn: enable many @IgnoreNative tests #1233

wants to merge 64 commits into from

Conversation

lauzadis
Copy link
Contributor

@lauzadis lauzadis commented Feb 4, 2025

Issue #

Description of changes

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

0marperez and others added 30 commits December 16, 2024 11:30
@0marperez
fix: create JMESPATH flattenIfPossible functions for Lists (#1169)
a4734c7
@aws-sdk-kotlin-ci
chore: release 1.3.30
7911a94
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.3.31-SNAPSHOT
7085c8a
@lauzadis
misc: enhance support for replayable instances of InputStream (#1197)
4575023
@aws-sdk-kotlin-ci
chore: release 1.3.31
b0a4bac
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.3.32-SNAPSHOT
80f4538
@lauzadis
fix: CBOR protocol test assertions / blob serialization (#1198)
e9d16a9
@lauzadis
fix: correctly serialize subset of shape's members when configured (#…
48849a1 
…1199)
@aws-sdk-kotlin-ci
chore: release 1.3.32
ded3a4b
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.3.33-SNAPSHOT
d97e8ba
@ianbotsf
chore: add *-main to branch workflows (#1209)
78f17f3
@ianbotsf
chore: add .kotlin/ to .gitignore (#1208)
d06ee0a
@0marperez
chore: smithy version bump (#1213)
d2e7c91
@aws-sdk-kotlin-ci
chore: release 1.3.33
5a2df8f
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.3.34-SNAPSHOT
003633b
@lauzadis
feat: add AuthTokenGenerator (#1212)
5f5ec8f
@aws-sdk-kotlin-ci
chore: release 1.3.34
3fe1b5e
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.3.35-SNAPSHOT
0bba308
@0marperez
misc: merge v1.4 into main (#1218)
52e4439
@0marperez
feat: support default checksums (#1191)
e0c25d6
@aws-sdk-kotlin-ci
chore: release 1.4.0
14e0958
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.4.1-SNAPSHOT
ed95d7b
@lauzadis
fix: add 0.9.x aws-crt-kotlin transform (#1220)
a4ace35
@lauzadis
fix: Ensure Host header is included when signing auth tokens (#1222)
15e5f0c
@aws-sdk-kotlin-ci
chore: release 1.4.1
447ac10
@aws-sdk-kotlin-ci
chore: bump snapshot version to 1.4.2-SNAPSHOT
0f8db44
@lauzadis
fix: address various failing protocol tests (#1223)
8b33693
@lauzadis
misc: re-enable kotlinWarningsAsErrors=true (#1224)
03badf9
@ianbotsf
fix: ignore hop-by-hop headers when signing requests (#1227)
9f44cdb
@aws-sdk-kotlin-ci
chore: release 1.4.2
97ac447
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@ianbotsf ianbotsf mentioned this pull request Feb 5, 2025
Merged
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@lauzadis lauzadis requested a review from ianbotsf February 6, 2025 16:06
0marperez
0marperez approved these changes Feb 6, 2025
View reviewed changes
Copy link
Contributor

@0marperez 0marperez left a comment

Choose a reason for hiding this comment

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

Question: Will you remove the toList workaround before merging?

runtime/protocol/aws-protocol-core/common/test/ClockSkewInterceptorTest.kt
Comment on lines +36 to +37
val clientTime = Instant.fromRfc5322("Fri, 6 Oct 2023 16:20:50 -0400")
val serverTime = Instant.fromRfc5322("Fri, 6 Oct 2023 16:20:50 -0400")
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!

@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025

Affected Artifacts

Changed in size
Artifact Pull Request (bytes) Latest Release (bytes) Delta (bytes) Delta (percentage)
http-test-jvm.jar 61,604 58,690 2,914 4.97%
aws-signing-default-jvm.jar 53,497 51,946 1,551 2.99%
runtime-core-jvm.jar 842,829 818,814 24,015 2.93%
crt-util-jvm.jar 21,451 20,952 499 2.38%
aws-signing-tests-jvm.jar 456,687 456,627 60 0.01%
test-suite-jvm.jar 96,930 97,206 -276 -0.28%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0marperez 0marperez 0marperez approved these changes

@ianbotsf ianbotsf Awaiting requested review from ianbotsf

Assignees
No one assigned
Labels
no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lauzadis @0marperez @ianbotsf @aws-sdk-kotlin-ci @xinsong-cui