Skip to content

Initial Commit#304

Open
postmeback wants to merge 1 commit into
xoriors:mainfrom
postmeback:unwrap-removal
Open

Initial Commit#304
postmeback wants to merge 1 commit into
xoriors:mainfrom
postmeback:unwrap-removal

Conversation

@postmeback
Copy link
Copy Markdown

Removed unnecessary unwraps

Fixes #279

Description

I have started going through the project. I have found out some places, where we don't need to use .unwrap(), we can use "?" or other options.

Fixes # (issue)

It is WIP. Suggestions are welcome.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist:

  • I have performed a self-review of my code
  • I have tested my code on different platforms (if applicable)
  • I have commented my code, particularly in hard-to-understand areas
  • I have added necessary documentation (if appropriate)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Screenshots (if applicable)

Add screenshots to illustrate changes if necessary.


Additional Context

Add any additional context or information here.

@postmeback postmeback requested a review from radumarias as a code owner May 31, 2025 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle errors insteaed of .unwrap()

1 participant