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=email :
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=email :
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=email :
PS1, Line 176:
trailing whitespace



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/41508?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Icba625c02a60d7e1f519b506a46bda5ded0537d3
Gerrit-Change-Number: 41508
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Tue, 25 Nov 2025 16:30:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to