Skip to content

Conversation

BartyBoi1128
Copy link
Contributor

@BartyBoi1128 BartyBoi1128 commented Feb 27, 2025

Created README and removed old code example for CWE-390.

BartKaras1128 and others added 2 commits February 27, 2025 11:23
Removed some trailing whitespace

Signed-off-by: BartyBoi1128 <[email protected]>
Copy link
Contributor

@myteron myteron left a comment

Choose a reason for hiding this comment

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

needs some updates

I thought this order made more sense than before, will discuss with Helge

Signed-off-by: BartyBoi1128 <[email protected]>
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.

Minor cosmetic suggestions. The most important part is replacing the code for example01.py in the markdown file.

BartyBoi1128 and others added 6 commits March 18, 2025 11:38
Should have been "in use" rather than "on use"

Co-authored-by: Hubert Daniszewski <[email protected]>
Signed-off-by: BartyBoi1128 <[email protected]>
Surrounding "except" in backticks

Co-authored-by: Hubert Daniszewski <[email protected]>
Signed-off-by: BartyBoi1128 <[email protected]>
Capitalizing "CTRL+C" to stay consistent

Co-authored-by: Hubert Daniszewski <[email protected]>
Signed-off-by: BartyBoi1128 <[email protected]>
Had the wrong Code Example in!

Co-authored-by: Hubert Daniszewski <[email protected]>
Signed-off-by: BartyBoi1128 <[email protected]>
Copy link
Contributor

@myteron myteron left a comment

Choose a reason for hiding this comment

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

Minor: we do not explain what the example01.py actually does. approved otherwise.

@BartKaras1128
Copy link
Contributor

Hi @myteron!
This looks good and can be merged now!

@myteron myteron merged commit 09cd05b into ossf:main Mar 28, 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.

4 participants