Skip to content

Conversation

scovetta
Copy link
Contributor

The code snippet incorrectly returns res when it should be returning ret. This doesn't really have an effect, since the sample demonstrates failing early.

The code snippet incorrectly returns `res` when it should be returning `ret`. This doesn't really have an effect, since the sample demonstrates failing early.

Signed-off-by: Michael Scovetta <[email protected]>
The same code is in both the README and compliant03.py. This change fixes the typo in compliant03.py.

Signed-off-by: Michael Scovetta <[email protected]>
@myteron
Copy link
Contributor

myteron commented Jan 14, 2025

@s19110 Hi Hubert, could you have a look at this? Thanks Helge

Copy link
Contributor

@s19110 s19110 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, ret should be the value that is returned here. Thank you for catching this typo.

@myteron myteron merged commit add9d61 into ossf:main Jan 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants