Skip to content

Conversation

@fabiencelier
Copy link
Contributor

Description

Can now be use with just dust-hive restart and select the env + service manually

image

or just specify the env and select the service

image

or specify all like before

image

Tests

manual tests

Risk

low

Deploy Plan

none

@vercel
Copy link

vercel bot commented Jan 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Review Updated (UTC)
playground Ignored Ignored Preview Jan 27, 2026 3:50pm
storybook Ignored Ignored Preview Jan 27, 2026 3:50pm

Request Review

Copy link
Contributor

@dust-agent dust-agent bot left a comment

Choose a reason for hiding this comment

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

Found 2 coding rule violations that should be addressed.

if (serviceArg) {
// Service provided via CLI argument
if (!isServiceName(serviceArg)) {
console.log(`\nServices: ${ALL_SERVICES.join(", ")}`);
Copy link
Contributor

Choose a reason for hiding this comment

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

[GEN8] Do not use console.log, console.error, etc. — always use the app logger

Direct calls to console.log are prohibited. Use the application logger instead:

Suggested change
console.log(`\nServices: ${ALL_SERVICES.join(", ")}`);
logger.info(`\nServices: ${ALL_SERVICES.join(", ")}`);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same as before

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.

2 participants