<p></p>
<p><b>@iosabi</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/15684#discussion_r563420773">tests/driver_si70xx/Makefile</a>:</p>
<pre style='color:#555'>> @@ -1,6 +1,8 @@
 include ../Makefile.tests_common
 
-# This test should also work with Si7006, Si7013 and Si7020 variants.
-USEMODULE += si7021
+# This test should also work with si7006, si7013, si7020 and si705x variants.
</pre>
<p>I was going with all lowercase when referring to the RIOT module name, and upper case when referring to the chip itself. Here since this is the string you need to pass to the Makefile I'd prefer to leave it lowercase.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/RIOT-OS/RIOT/pull/15684#discussion_r563420773">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYCJOMBACQJA5QYNEHLS3TDHJANCNFSM4VCM3V3A">unsubscribe</a>.<img src="https://github.com/notifications/beacon/ABE7WYDVQNEXW6SLTWJ6HZLS3TDHJA5CNFSM4VCM3V3KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOEJDBTZQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/RIOT-OS/RIOT/pull/15684#discussion_r563420773",
"url": "https://github.com/RIOT-OS/RIOT/pull/15684#discussion_r563420773",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>