Skip to content

Reorganize the module into folders with v.mod subdirs - #7

Merged
davlgd merged 3 commits into
vlang-community:mainfrom
davlgd:davlgd-organize-folders
Sep 24, 2026
Merged

davlgd merged 3 commits into
vlang-community:mainfrom
davlgd:davlgd-organize-folders

Conversation

@davlgd

@davlgd davlgd commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

@Jengro777 sorry about that. Since V dropped the virtual src/ directory, I've gotten into the habit of keeping modules flat, and I overdid it here. Following your comment, this brings a hierarchy back:

kdl.v        data model and module docs
parser/      parser.v, scanner.v, strings.v
writer/      writer.v
marshal/     marshal.v (public API), schema.v, encode.v, decode.v
tests/       all tests, plus the vendored test_cases/
examples/

The folders are listed as subdirs in v.mod, the mechanism that replaced src/ (vlang/v@5518bfc). Every file stays in module kdl, so the API is unchanged, and the tests now only use the public API. The source files are moved as they are, with no code changes.

One caveat: v doc doesn't follow subdirs yet, so symbols from the subfolders are missing from the generated docs. I can send a fix upstream if you're OK with this layout.

Copilot AI balanced review requested due to automatic review settings September 23, 2026 17:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Jengro777

Copy link
Copy Markdown
Member

You can use AI fit's confirmed, you can merge the PR your

The kdl library itself is pretty small, so the main focus going forward is on stability and releasing it to the vpm site

@davlgd

davlgd commented Sep 24, 2026

Copy link
Copy Markdown
Contributor Author

you can merge the PR

I don't have rights to do it

@Jengro777

Copy link
Copy Markdown
Member

you can merge the PR

I don't have rights to do it

Try again

@davlgd
davlgd merged commit 9707a81 into vlang-community:main Sep 24, 2026
4 checks passed
@davlgd
davlgd deleted the davlgd-organize-folders branch September 24, 2026 12:54
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.

3 participants