Francesco Romani

Results 485 comments of Francesco Romani

> We used to add `vendor/` intentionally b/c of some code (zz_openapi*.go) are excluded in git, but needed to make codegen functional. So it ends up developer having to locally...

@jdtuhui it seems there is a growing consensus towards merging this PR. Could you please rebase on top of current master branch?

thanks for adding the helper! I think (but my experience with DRA is very limited) this is a very nice addition. I have a minor comment about the code layout....

just curious why close? is there a better direction for this work?

> > just curious why close? is there a better direction for this work? > > @ffromani all I wanted to do is push new changes and when I did...

/cc @haircommander hey Peter could you please review the CRI side and/or suggest CRI reviewers?