Skip to content

refactor: replace unwrap() with if let Some() in snapshot parsing#1563

Merged
kodiakhq[bot] merged 1 commit intomainfrom
rd/clippy
Jan 26, 2026
Merged

refactor: replace unwrap() with if let Some() in snapshot parsing#1563
kodiakhq[bot] merged 1 commit intomainfrom
rd/clippy

Conversation

@erka
Copy link
Collaborator

@erka erka commented Jan 26, 2026

So clippy is happy

@erka erka requested a review from a team as a code owner January 26, 2026 19:55
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. chore labels Jan 26, 2026
@dosubot
Copy link

dosubot bot commented Jan 26, 2026

Related Documentation

Checked 2 published document(s) in 0 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jan 26, 2026
So clippy is happy and update test

Signed-off-by: Roman Dmytrenko <[email protected]>
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.40%. Comparing base (f701007) to head (9e2ecb5).
⚠️ Report is 847 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1563      +/-   ##
==========================================
- Coverage   80.36%   75.40%   -4.96%     
==========================================
  Files           8       10       +2     
  Lines        4165     4473     +308     
==========================================
+ Hits         3347     3373      +26     
- Misses        818     1100     +282     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@erka erka added the automerge label Jan 26, 2026
Copy link
Contributor

@markphelps markphelps left a comment

Choose a reason for hiding this comment

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

nice one

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 26, 2026
@kodiakhq kodiakhq bot merged commit c7418bd into main Jan 26, 2026
38 checks passed
@kodiakhq kodiakhq bot deleted the rd/clippy branch January 26, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge chore lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants