lakeFS
lakeFS copied to clipboard
ariels' version of: Upgrade version of `oapi-codegen`
ariels' version of #7493. This one is more for working and linking than it is for pulling!
No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged
Looks like the current failures are due to kin-openapi
changes in between the versions we're using - I can see pkg/api/tmpl/inline.tmpl
may need an update, but once updating it it doesn't regenerate any of the templates we've got
Looks like the current failures are due to
kin-openapi
changes in between the versions we're using - I can seepkg/api/tmpl/inline.tmpl
may need an update, but once updating it it doesn't regenerate any of the templates we've got
Yeah, I figured those out. Now it's broken because the new codegen the OpenAPI 3.1 interpretation that forbids DELETE with a body. Can I override that setting somehow?
@arielshaqed the reason for the failures was due to the incorrect placement of the template files. It changed a lot since the old 1.5.6 days...
Mind the change under the: pkg/api/apigen/tmpl
path
🎊 PR Preview 9ebd11895d90dae79f1776f56b245603275a3c62 has been successfully built and deployed to https://treeverse-lakeFS-preview-pr-7532.surge.sh
:clock1: Build time: 0.012s
🤖 By surge-preview
E2E Test Results - DynamoDB Local - Local Block Adapter
This PR is now marked as stale after 30 days of inactivity, and will be closed soon. To keep it, mark it with the "no stale" label.