Skip to content
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

Replace Go model generation for tekton #6422

Closed
manusa opened this issue Oct 7, 2024 · 0 comments · Fixed by #6449
Closed

Replace Go model generation for tekton #6422

manusa opened this issue Oct 7, 2024 · 0 comments · Fixed by #6449
Assignees
Labels
Milestone

Comments

@manusa
Copy link
Member

manusa commented Oct 7, 2024

Description

Part of #6130

Replace model generation for tekton module with new OpenAPI approach.

Note

For extension models there are going to be multiple conflicts with the previously generated code.

Tasks

  • Remove build-helper-maven-plugin
  • Remove maven-antrun-plugin from the generate profile
  • Remove jsonschema2pojo-maven-plugin from the generate profile
  • Consolidate all extension models to the same module (certmanager, tekton, and so on all have multiple model generator modules)
  • Configure the openapi-model-generator-maven-plugin plugin to generate the specific classes.
  • Remove the go-related files and directories (Makefile, cmd)
  • Remove invocation from the generateModel.sh script
@manusa manusa self-assigned this Oct 14, 2024
@manusa manusa moved this to In Progress in Eclipse JKube Oct 14, 2024
@manusa manusa added this to the 7.0.0 milestone Oct 14, 2024 — with automated-tasks
@github-project-automation github-project-automation bot moved this from Review to Done in Eclipse JKube Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant