-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cli: send bot config to backend in botonic deploy #BLT-1008 #2884
cli: send bot config to backend in botonic deploy #BLT-1008 #2884
Conversation
Test Results30 tests ±0 26 ✅ ±0 35s ⏱️ ±0s For more details on these failures, see this check. Results for commit 2099368. ± Comparison against base commit fd86324. ♻️ This comment has been updated with latest results. |
const exec = util.promisify(childProcess.exec) | ||
const { stdout } = await exec(command) | ||
return stdout.trim() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we try with execSync?
const { execSync } = require('child_process');
try {
const stdout = execSync(command, { encoding: 'utf-8' });
console.log(stdout);
} catch (error) {
console.error('Error:', error);
}
301f823
to
9845ac8
Compare
…s at the same time
## Description Refactor the deploy file to always use async await instead of chaining asyncorn functions with callbacks inside the then function.
1a74c86
to
2099368
Compare
Description
When deploying a bot the cli will get bot config from the @Botonic packages installed, npm and node version .
This config is send to backend in deploy endpoint.
Context
This config will be used to read bot information from the flow builder frontend.
Approach taken / Explain the design
PRs merged in this PR: