Skip to content

tests_with_kvdb.sh: fix comparison in shell#34

Merged
neteler merged 1 commit intomainfrom
shell_fix_comparison
Jan 6, 2026
Merged

tests_with_kvdb.sh: fix comparison in shell#34
neteler merged 1 commit intomainfrom
shell_fix_comparison

Conversation

@neteler
Copy link
Member

@neteler neteler commented Jan 6, 2026

Comparison in shell is done with single = to be portable.

Comparison in shell is done with single `=` to be portable.
@neteler neteler requested a review from mmacata January 6, 2026 15:09
@neteler neteler self-assigned this Jan 6, 2026
@neteler neteler added the bug Something isn't working label Jan 6, 2026
@neteler
Copy link
Member Author

neteler commented Jan 6, 2026

Note, the linting error is unrelated:

Run flake8 --config=.flake8 --count --statistics --show-source .
./tests/test_resource_base.py:93:9: F824 `global kvdb_pid` is unused: name is never assigned in scope
        global kvdb_pid
        ^
1     F824 `global kvdb_pid` is unused: name is never assigned in scope

@neteler neteler merged commit 9668f47 into main Jan 6, 2026
11 of 12 checks passed
@neteler neteler deleted the shell_fix_comparison branch January 6, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants