Fix broken skills (additionalDirectories) in 0.130.0 #156
Fix broken skills (additionalDirectories) in 0.130.0 #156NikolaiSviridov wants to merge 8 commits into
Conversation
|
Why this is done on top of update to 0.130.0? I would expect that we first change skills to new implementation and the do codex update (probably to latest 0.132.0) and check that all tests are still green. |
|
@AlexandrSuhinin skills were broken in 0.130.0 and there is a new logic related to skill handling in it (recomended by openai). So, I wanted to merge update 0.130.0 and update to 0.132.0 after. Can move it to 0.132.0 |
|
Please add e2e tests if they are missing. |
|
So there is no marketplace API in 0.128.0 version and you're dependent on update to 0.130.0+? |
|
Also I don't really understand if it's a bug, but this review doesn't include all related changes. |
|
Please rebase this branch on master if marketplace events are supported in 0.128.0. |
5f49e5e to
17f7fee
Compare
17f7fee to
c7b6f36
Compare
No description provided.