Jenkins Builder has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/41508?usp=email )
Change subject: card_key_provider: add PostgreSQL support ......................................................................
Patch Set 1:
(3 comments)
File contrib/csv-to-pgsql.py:
Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/pysim/+/41508/comment/dbdc53e1_0538a54e?usp=ema... : PS1, Line 129: Get a list of all columns availabe in the current table scheme. 'availabe' may be misspelled - perhaps 'available'?
File docs/card-key-provider.rst:
Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/pysim/+/41508/comment/c65fb3b8_cfe1c7d5?usp=ema... : PS1, Line 144: assume that you didn't get the Global Plattform keys from your card vendor for 'Plattform' may be misspelled - perhaps 'Platform'?
Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/pysim/+/41508/comment/af57878f_3376d5e9?usp=ema... : PS1, Line 176: trailing whitespace