networknt / networknt/light-codegen

several issues that need to be discussed to find better solutions

Open
#307 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Java
Stars
49
Forks
25
PR merge metrics
No merged PRs in 30d

Description

  1. 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.

  2. 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.

  3. Is there a way to set the default value for singletons in the service.yml file. If yes, we need to do that.

  4. 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

  5. Make the values.yml generation as default to encourage users to use it. It can be optionally turned off from config.json?

  6. More to add during the discussion.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.