fixed issue #18

[inyourface34456]

changed the controller to an option

[Amjad50]

A small improvement here,
you can use

// didn't test the code, but the idea here
let Some(mut gilrs) = self.gilrs else {
    return;
};

then afterwards, you can use girls without the need for as_mut().unwrap()

[inyourface34456]

Thanks for the suggestion, will implement

EDIT: Unforuntly, the girls object does not implement copy, so it cant be moved out of the Some variant. Good idea nonetheless.

[Amjad50]

You can use ref

[Amjad50]

Also please run cargo fmt before the next commit, looks like its failing the CI.
Also check cargo clippy

[inyourface34456]

Also please run cargo fmt before the next commit, looks like its failing the CI. Also check cargo clippy

Fixed all of the issues.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.81%. Comparing base (f254082) to head (eb5633d).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   81.81%   81.81%           
=======================================
  Files          37       37           
  Lines        5352     5352           
=======================================
  Hits         4379     4379           
  Misses        973      973           

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

[Amjad50]

Thanks for this. Great