Skip to content

feat(moduleconfig): add shouldWrapValues setting - #325

Merged
elpete merged 1 commit into
nextfrom
feature/issue-309-should-wrap-values
Aug 12, 2026
Merged

feat(moduleconfig): add shouldWrapValues setting#325
elpete merged 1 commit into
nextfrom
feature/issue-309-should-wrap-values

Conversation

@elpete

@elpete elpete commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Supersedes #309 because the contributor fork does not allow maintainer edits.

Preserves the original contributor commit authorship while addressing review feedback:

  • exposes shouldWrapValues as a module setting
  • keeps AutoDiscover lazy instead of querying database metadata during module load
  • applies the configured setting when the concrete grammar is discovered
  • verifies lazy discovery and setting propagation
  • removes the manually generated changelog entry

Closes #309.

@elpete
elpete changed the base branch from main to next August 12, 2026 22:03
@elpete
elpete force-pushed the feature/issue-309-should-wrap-values branch from 323f287 to d195335 Compare August 12, 2026 22:06
@elpete
elpete merged commit d6762ce into next Aug 12, 2026
18 checks passed
@elpete
elpete deleted the feature/issue-309-should-wrap-values branch August 12, 2026 22:17
elpete added a commit that referenced this pull request Aug 12, 2026
Co-authored-by: Karlin [bot] <karlin@inleague.org>
elpete added a commit that referenced this pull request Aug 13, 2026
Co-authored-by: Karlin [bot] <karlin@inleague.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant