networknt / networknt/light-codegen
several issues that need to be discussed to find better solutions
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 49
- Forks
- 25
- PR merge metrics
- No merged PRs in 30d
Description
-
We cannot ask users to add this exclude for the parameterizer in the config.json. In my opinion, we might need to put the parameters into the template instead of processing the config files.
-
We need to find a way to move the values.yml into the config folder in the generated project. However, we cannot do that as the service.yml need to set the value for the service.singletons.
-
Is there a way to set the default value for singletons in the service.yml file. If yes, we need to do that.
-
We need to think about the existing users with existing config.json when we change the light-codegen. That means we should always have a default value for the new flags we add to the config.json
-
Make the values.yml generation as default to encourage users to use it. It can be optionally turned off from config.json?
-
More to add during the discussion.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing how light-codegen currently handles config.json, values.yml, and service.yml. Resolve the listed compatibility, singleton-default, and values.yml-generation questions with maintainers before implementation. Done means the open-ended discussion has an agreed scope and a clear design for existing and newly generated projects.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100