We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Group
Def<Settings>
There is no reason to store a reference to Settings. Rather, Group only needs access to root_path.
Settings
root_path
Arc
Device::init_log()
Log
The text was updated successfully, but these errors were encountered:
Group should not store Def<Settings> directly #168
5789158
Successfully merging a pull request may close this issue.
There is no reason to store a reference to
Settings
. Rather,Group
only needs access toroot_path
.root_path
should be behindArc
inSettings
#171Device::init_log()
should not accept parameters #169Log
should store reference toroot_path
, notSettings
#170The text was updated successfully, but these errors were encountered: