Skip to content

workspace: crate directory grouping - #426

Merged
npry merged 1 commit into
mainfrom
npry/refactor.grouping
Oct 7, 2026
Merged

npry merged 1 commit into
mainfrom
npry/refactor.grouping

Conversation

@npry

@npry npry commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Re-group the crates into directories roughly following what we discussed last week in sync, including nesting into a crates/ dir.

This just moves the crates and updates CI to point to the new locations; there are a few more merges/refactors I'm considering (e.g. how/where to merge shared types from derp, control, and probably the capabilities crates), but I wanted to keep this change minimal.

@npry
npry force-pushed the npry/refactor.grouping branch 3 times, most recently from 179cbdb to 2dbe7b7 Compare September 23, 2026 20:34
@nrc

nrc commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

some naming nits opinions:

binding -> bindings
control -> control_plane
datastructure -> data_structures
kv -> kv_store
proto -> protos
tool -> tools
util -> utils

@npry
npry force-pushed the npry/refactor.grouping branch 8 times, most recently from 3521641 to 42bbcca Compare September 29, 2026 17:28
@npry
npry marked this pull request as ready for review September 29, 2026 17:31
@npry
npry force-pushed the npry/refactor.grouping branch from 42bbcca to b561abf Compare September 29, 2026 17:40
The top-level directory was pretty messy: group crates into directories
by function.

Signed-off-by: Nathan Perry <nathan@tailscale.com>
Change-Id: Ieed50c98d4473288adb5f4a8f8abfc586a6a6964
@npry
npry force-pushed the npry/refactor.grouping branch from b561abf to 38b80f3 Compare September 30, 2026 13:57
@npry npry changed the title workspace: candidate directory grouping workspace: crate directory grouping Oct 7, 2026
@npry
npry merged commit 38b80f3 into main Oct 7, 2026
23 checks passed
@npry
npry deleted the npry/refactor.grouping branch October 7, 2026 18:19
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