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

Add a tool sdk-lockfiles to audit lockfiles #3827

Merged
merged 3 commits into from
Sep 12, 2024
Merged

Conversation

ysaito1001
Copy link
Contributor

Motivation and Context

See README

A later PR will use this tool to enforce the check (likely during the execution of pre-commit-hooks).

Testing

  • Added unit tests for the tool
  • Ran the tool against currently checked-in lockflies in the main branch and ensured the SDK lockfile contained dependencies listed in the other runtime lockfiles.

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

Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

@ysaito1001 ysaito1001 marked this pull request as ready for review September 11, 2024 17:12
@ysaito1001 ysaito1001 requested review from a team as code owners September 11, 2024 17:12
Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

@ysaito1001 ysaito1001 added this pull request to the merge queue Sep 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 12, 2024
@ysaito1001 ysaito1001 added this pull request to the merge queue Sep 12, 2024
Merged via the queue into main with commit 4230687 Sep 12, 2024
44 checks passed
@ysaito1001 ysaito1001 deleted the ysaito/audit-lockfiles branch September 12, 2024 23:42
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/README.md Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
tools/ci-build/sdk-lockfiles/src/audit.rs Show resolved Hide resolved
ysaito1001 added a commit that referenced this pull request Sep 16, 2024
ysaito1001 added a commit that referenced this pull request Sep 16, 2024
ysaito1001 added a commit that referenced this pull request Sep 16, 2024
ysaito1001 added a commit that referenced this pull request Sep 16, 2024
ysaito1001 added a commit that referenced this pull request Sep 16, 2024
This commit addresses #3827 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants