You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ma...@apache.org on 2024/03/21 18:46:11 UTC
(superset) 01/02: fix 'supersetbot docker' preset argument
This is an automated email from the ASF dual-hosted git repository.
maximebeauchemin pushed a commit to branch fix_docker_base
in repository https://gitbox.apache.org/repos/asf/superset.git
commit 4fdf5a52da9743bdbf8830d67d601bfb515de90a
Author: Maxime Beauchemin <ma...@gmail.com>
AuthorDate: Thu Mar 21 11:36:33 2024 -0700
fix 'supersetbot docker' preset argument
---
.github/supersetbot/src/cli.js | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/.github/supersetbot/src/cli.js b/.github/supersetbot/src/cli.js
index b8344d6ac3..4b4612ee97 100755
--- a/.github/supersetbot/src/cli.js
+++ b/.github/supersetbot/src/cli.js
@@ -154,16 +154,16 @@ export default function getCLI(context) {
program.command('docker')
.description('Generates/run docker build commands use in CI')
- .option('-t, --preset', 'Build preset', /^(lean|dev|dockerize|websocket|py310|ci)$/i, 'lean')
+ .option('-t, --preset <preset>', 'Build preset', /^(lean|dev|dockerize|websocket|py310|ci)$/i, 'lean')
.option('-c, --context <context>', 'Build context', /^(push|pull_request|release)$/i, 'local')
.option('-r, --context-ref <ref>', 'Reference to the PR, release, or branch')
.option('-p, --platform <platform...>', 'Platforms (multiple values allowed)')
.option('-f, --force-latest', 'Force the "latest" tag on the release')
.option('-v, --verbose', 'Print more info')
- .action(function (preset) {
- const opts = context.processOptions(this);
+ .action(function () {
+ const opts = context.processOptions(this, ['preset']);
opts.platform = opts.platform || ['linux/arm64'];
- const cmd = docker.getDockerCommand({ preset, ...opts });
+ const cmd = docker.getDockerCommand({ ...opts });
context.log(cmd);
if (!opts.dryRun) {
utils.runShellCommand(cmd, false);