Closed VukW closed 3 months ago
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.10%. Comparing base (
2a5707b
) to head (c1742bb
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
old version prohibited first word to finish with
s
char, that is quite a weirdFixes #ISSUE_NUMBER
Proposed Changes
(?<!s)
meansthere should not be an "s" char
, that's quite dumb. Removing this restrictionChecklist
CONTRIBUTING
guide has been followed.typing
is used to provide type hints, including and not limited to usingOptional
if a variable has a pre-defined value).pip install
step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].