Skip to content
This repository was archived by the owner on Dec 25, 2024. It is now read-only.

v3 java package consolidation, folder cleanup, docker image size reduction #189

Merged
merged 43 commits into from
Jul 22, 2023

Conversation

spacether
Copy link
Contributor

@spacether spacether commented Jul 20, 2023

v3 java package consolidation

  • this projects used 3 separate packages under the hood which is an unneeded amount of complexity
  • merged 3 packages into one
  • refactored java code to be better organized
  • Generator -> GeneratorRunner
  • CodegenConfig -> Generator
  • some unneeded root level folders deleted
  • docker image reduced from 863 mb to 316 mb by using multi stage dockerfle

PR checklist

  • Read the contribution guidelines.
  • Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
  • Run the following to build the project and update samples:
    ./mvnw clean package 
    ./bin/generate-samples.sh
    ./bin/utils/export_docs_generators.sh
    
    Commit all changed files.
    This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
    These must match the expectations made by your contribution.
    You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/python*.
    For Windows users, please run the script in Git BASH.

@spacether spacether changed the title v3 java package consolidation v3 java package consolidation + folder cleanup Jul 21, 2023
@spacether spacether changed the title v3 java package consolidation + folder cleanup v3 java package consolidation, folder cleanup, docker image size reduction Jul 22, 2023
@spacether spacether merged commit 0651c7e into 3_0_0 Jul 22, 2023
@spacether spacether deleted the v3_java_package_consolidation branch July 22, 2023 06:00
spacether added a commit that referenced this pull request Aug 16, 2023
…ction (#189)

* Merges core into generator

* Refactors codegen module

* More refactoring

* Renames Generator as GeneratorRunner, CodegenConfig renamed to Generator

* Removes common subpackage

* Reverts validation public changes

* Moves java python client test file

* Moves meta into generators and renames it generatormetadata

* Removes example module

* Renames codegenerator to generatorrunner

* Moves models into generators and renames to openapimodels

* Moves ignore int generator, removes unneeded classes for python generator

* Moves ignore moduel into generatorrunner

* validation module refactored to move external imports to its root level

* Moves java test file

* Removes cli module

* Refactors cli into clicommands module

* Collapses cli module into remaining module

* Updates dockerfile

* Removes broken links from pom, removes qos library, increase slf4j version

* Removes logToStderr

* Removes unused imports

* Fixes arch tests

* Removes sonatype from pom

* Removes logback xml exclusion

* Fixes groupId and package info

* Fixes mainClass definition

* Adds shade plugin, fixes jar running

* Fixes version info in docker

* Removes volume

* Adds multi stage docker image

* Reduces image size further

* Gets checkstyle working

* Removes scripts modules and website

* Removes website references

* Moves circleci parallel file into .circleci folder

* turns filtering off, adds pom comment for a versin upgrade

* Removes CI folder

* Changes suffix from project back to cli

* Removes many module and languages references

* Ensure up to date run

* Turns on filtering for docker verison command

* Actually fixes resource filtering
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant