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

Consolidate config schema and default values #18

Merged
merged 1 commit into from
Feb 25, 2023

Conversation

stefanprodan
Copy link
Owner

@stefanprodan stefanprodan commented Feb 25, 2023

Changes to module structure:

  • Make templates/config.cue the place where module authors can define the config schema, constraints and default values.
  • Make values.cue a simple placeholder for user-supplied values.
  • Set the values schema in timoni.cue for module authors to understand how the user-supplied are validated.
  • Add test values and CUE tooling for module authors.

@stefanprodan stefanprodan added area/docs Improvements or additions to documentation area/engine CUE engine related issues and pull requests labels Feb 25, 2023
@stefanprodan stefanprodan merged commit 4eeda97 into main Feb 25, 2023
@stefanprodan stefanprodan deleted the consolidate-defaults branch February 25, 2023 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Improvements or additions to documentation area/engine CUE engine related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant