search

knative / pkg

Knative common packages
Apache License 2.0
259 stars 331 forks source link

Rewrite the Knative code-gen to a Go-native tool #2809

Open cardil opened 1 year ago

cardil commented 1 year ago

Expected Behavior

To fix https://github.com/knative/infra/issues/134 we need to remove the shell wrapper, so the Knative projects could run the code-gen in go run ... native way.

Actual Behavior

Current code gen has a shell wrapper. Because of it Knative projects which use this code gen require the vendor directory.

dprotaso commented 1 year ago

Note for a bunch of the codegen we do - it could be generics now

Priyansurout commented 1 year ago

/assign

github-actions[bot] commented 10 months ago

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

cardil commented 4 months ago

/reopen /remove-lifecycle stale /triage accepted

knative-prow[bot] commented 4 months ago

@cardil: Reopened this issue.

In response to [this](https://github.com/knative/pkg/issues/2809#issuecomment-2196876580): >/reopen >/remove-lifecycle stale >/triage accepted Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.