Skip to content

update-deps.yml: fixed failure when running workflow#1821

Merged
larsewi merged 2 commits intocfengine:masterfrom
larsewi:dollar
Jul 28, 2025
Merged

update-deps.yml: fixed failure when running workflow#1821
larsewi merged 2 commits intocfengine:masterfrom
larsewi:dollar

Conversation

@larsewi
Copy link
Copy Markdown
Contributor

@larsewi larsewi commented Jul 28, 2025

No description provided.

Previously attempted to fix workflow in
cfengine#1819 by adding missing
dollar sign. However, it seems like it still does not work properly.
Looks like the conditional argument evaluated to the string `false`.

```
Run python3 /tmp/update-deps.py --debug --bump=minor false
usage: update-deps.py [-h] [--debug] [--bump {major,minor,patch}]
                      [--skip PACKAGE VERSION] [--root ROOT] [--jdk21]
update-deps.py: error: unrecognized arguments: false
```

Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Comment thread .github/workflows/update-deps.yml
Co-authored-by: Craig Comstock <craig.comstock@northern.tech>
@larsewi larsewi merged commit 48e1c36 into cfengine:master Jul 28, 2025
1 of 2 checks passed
@larsewi larsewi deleted the dollar branch October 2, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants