[riot-notifications] [RIOT-OS/RIOT] WIP: tests/ieee802154_hal: extend radio hal test (#15761)

Martine Lenders notifications at github.com
Mon Jun 7 14:11:05 CEST 2021


@miri64 commented on this pull request.



> @@ -0,0 +1,98 @@
+# Copyright (C) 2019-20 Freie Universität Berlin
+#
+# This file is subject to the terms and conditions of the GNU Lesser
+# General Public License v2.1. See the file LICENSE in the top level
+# directory for more details.
+
+import re
+
+from riotctrl.shell import ShellInteraction
+
+# ==== ShellInteractions ====
+
+class Config_phy(ShellInteraction):

Please use either `snail_case` or `CamelCase` and do not mix them. For class names PEP 8 recomments `CamelCase`.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15761#pullrequestreview-677331640
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210607/cf584ea8/attachment-0001.htm>


More information about the notifications mailing list