Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Deal with standalone packages #352

Merged
merged 4 commits into from
Feb 12, 2025
Merged

Deal with standalone packages #352

merged 4 commits into from
Feb 12, 2025

Conversation

timotheeguerin
Copy link
Owner

No description provided.

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 11 changed files in this pull request and generated no comments.

Files not reviewed (6)
  • packages/chronus/src/dependency-graph/get-dependency-graph.ts: Evaluated as low risk
  • packages/chronus/src/workspace-manager/pnpm.ts: Evaluated as low risk
  • packages/chronus/src/utils/node-host.ts: Evaluated as low risk
  • packages/chronus/src/workspace-manager/utils.ts: Evaluated as low risk
  • packages/chronus/src/config/parse.ts: Evaluated as low risk
  • packages/chronus/src/config/types.ts: Evaluated as low risk
Comments suppressed due to low confidence (1)

packages/chronus/src/workspace/load.ts:24

  • The new behavior introduced by handling standalone packages should be covered by tests to ensure it works correctly.
const additionalPackages: Package[] = await loadStandalonePackages(host, config);
Copy link
Contributor

github-actions bot commented Feb 12, 2025

All changed packages have been documented.

  • @chronus/chronus
Show changes

@chronus/chronus - fix ✏️

Correctly respect negate patterns in pnpm workspace packages list

@chronus/chronus - feature ✏️

Add new additionalPackages entry which can be used to give path to some additional packages that are not part of the main workspace and should still be versioned.

@timotheeguerin timotheeguerin merged commit 5193006 into main Feb 12, 2025
6 checks passed
@timotheeguerin timotheeguerin deleted the standalone-packages branch February 12, 2025 22:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant