-
Notifications
You must be signed in to change notification settings - Fork 5.5k
SDK CI #4483
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
SDK CI #4483
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ 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
…K test in workflow
The CI workflow no longer runs or checks sdk/typescript tests and excludes its filter from change detection, streamlining the Rust CI jobs.
- Add eslint-plugin-jest v29.0.1 to dependencies and pnpm lock - Update .eslintrc.json to include jest/globals env and jest plugin - Improves linting support for Jest test files
- Remove .eslintrc.json in favor of flat config with eslint.config.js using typescript-eslint preset. - Drop unused eslint-plugin-import and eslint-import-resolver-typescript dependencies. - Update scripts to invoke eslint via pnpm for monorepo compatibility. - Update pnpm-lock.yaml to reflect new dependency graph.
Build debug codex in SDK configuration