Skip to content

[Refactor]: View.getClasses() from Collection to to Stream #1027#1030

Merged
swissiety merged 6 commits intosoot-oss:developfrom
JebeKerman:refactor/1027-view-get-classes
Aug 17, 2024
Merged

[Refactor]: View.getClasses() from Collection to to Stream #1027#1030
swissiety merged 6 commits intosoot-oss:developfrom
JebeKerman:refactor/1027-view-get-classes

Conversation

@JebeKerman
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Collaborator

@swissiety swissiety left a comment

Choose a reason for hiding this comment

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

thx for your work @JebeKerman !

Copy link
Copy Markdown
Collaborator

@swissiety swissiety left a comment

Choose a reason for hiding this comment

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

well done! That simplified a lot.

@JebeKerman
Copy link
Copy Markdown
Author

Very cool, thanks for the help :)

@swissiety swissiety merged commit 753427b into soot-oss:develop Aug 17, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.95%. Comparing base (f94b4fa) to head (199ebe8).
Report is 25 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #1030      +/-   ##
=============================================
+ Coverage      70.24%   71.95%   +1.71%     
- Complexity      4199     4454     +255     
=============================================
  Files            316      367      +51     
  Lines          15748    16377     +629     
  Branches        2699     2740      +41     
=============================================
+ Hits           11062    11784     +722     
+ Misses          3812     3688     -124     
- Partials         874      905      +31     

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

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.

3 participants