Add credential vault SDKs and e2e coverage#1023
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ab0b6735ee
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d5310a037d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 441b40f2c7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…tch block' Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Summary
Validation
bash -n scripts/credential-vault-e2e-target.sh scripts/python-e2e.sh scripts/go-e2e.sh scripts/javascript-e2e.sh scripts/java-e2e.sh.github/workflows/real-e2e.ymlwithyaml.safe_loaduv run ruff check tests/test_credential_vault_e2e.pyuv run pytest tests/test_credential_vault_e2e.py -qGOCACHE=/private/tmp/opensandbox-go-build-cache go test ./... -run TestCredentialVault -count=1 -vcorepack pnpm exec tsc --noEmit --target ES2022 --module NodeNext --moduleResolution NodeNext --allowImportingTsExtensions --strict tests/test_credential_vault_e2e.test.tscorepack pnpm exec eslint tests/test_credential_vault_e2e.test.tscorepack pnpm exec vitest run tests/test_credential_vault_e2e.test.tsmavenLocalwith JDK 17./gradlew test --tests com.alibaba.opensandbox.e2e.CredentialVaultE2ETest -PskipCodeInterpreterE2E=true --rerun-tasksgit diff --checkC# SDK unit tests were not run locally because
dotnetis not installed in this environment.