-
Notifications
You must be signed in to change notification settings - Fork 53
feat: support codex cli #281
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
Conversation
@matifali PR ready for review |
@hugodutka @DevelopmentCats, please review and let me know if this needs any changes :) |
@hugodutka thanks for the review, committed the suggested changes. |
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.
Great work, just some nits. I expect we'll merge after the next review.
Co-authored-by: Hugo Dutka <dutkahugo@gmail.com>
Co-authored-by: Hugo Dutka <dutkahugo@gmail.com>
Co-authored-by: Hugo Dutka <dutkahugo@gmail.com>
Co-authored-by: Hugo Dutka <dutkahugo@gmail.com>
@hugodutka done ! |
@DevelopmentCats please let me know if this needs any changes. |
Sorry my server that I host was having some problems over the weekend. I will make sure that I pull this and test it today so we can get it merged :) |
… to Codex module - Introduced new variables for sandbox configuration: sandbox_mode, approval_policy, and network_access. - Updated README to reflect the new variables and their usage. - Modified install and start scripts to utilize the new variables for better configuration management.
Closes #236
/claim #236
Description
https://www.loom.com/share/4a39d1c2e2324cb2b6f90c1ad6893602?sid=d5211d2c-a2cd-41ae-8580-15ac686756f2
Type of Change
Module Information
Path:
registry/[namespace]/modules/[module-name]
New version:
v1.0.0
Breaking change: [ ] Yes [ ] No
Testing & Validation
bun test
)bun run fmt
)Related Issues