Skip to content

move packages out of root#148

Merged
shanedg merged 6 commits into
mainfrom
dev/move-projects-out-of-repo-root
Jul 8, 2026
Merged

move packages out of root#148
shanedg merged 6 commits into
mainfrom
dev/move-projects-out-of-repo-root

Conversation

@shanedg

@shanedg shanedg commented Jun 30, 2026

Copy link
Copy Markdown
Owner

No description provided.

@shanedg shanedg force-pushed the dev/move-projects-out-of-repo-root branch 2 times, most recently from 308e8d9 to a5979a8 Compare June 30, 2026 08:44
Comment thread common/config/rush/command-line.json Outdated
@shanedg shanedg force-pushed the dev/move-projects-out-of-repo-root branch from 40074d8 to 363aaee Compare July 1, 2026 07:51
@shanedg shanedg force-pushed the dev/move-projects-out-of-repo-root branch from 363aaee to 94b0503 Compare July 8, 2026 08:32
shanedg added 6 commits July 8, 2026 12:17
Nest all packages under "/projects" directory to declutter repo root.
Avoids duplication of eslint ecma version.
Avoid duplication of supported node version across package.json engines.
Make plop a regular dependency instead of a peer.
@shanedg shanedg force-pushed the dev/move-projects-out-of-repo-root branch from 94b0503 to 95b9613 Compare July 8, 2026 16:36
Comment thread .vscode/settings.json
"markdownlint-config",
"scaffold",
"typescript"
"projects/*"

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

nesting projects under new directory makes it more convenient to define working directories via glob

path: join(relativePathToRepoRoot, '.vscode', 'settings.json'),
pattern: /"eslint.workingDirectories": \[/,
template: ' "{{path}}",',
},

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

no longer necessary with working directories glob!


const { configs: nodeConfigs } = node;

const supportedNodeVersionRange = '>=24.16.0';

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

single place to adjust node version (for linting purposes) instead of every package.json#engines field

@shanedg shanedg marked this pull request as ready for review July 8, 2026 16:43
@shanedg shanedg merged commit 04f2218 into main Jul 8, 2026
6 checks passed
@shanedg shanedg deleted the dev/move-projects-out-of-repo-root branch July 8, 2026 16:43
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