Skip to content
Snippets Groups Projects

Remove the GetVariables Trait

Merged AdminXVII requested to merge AdminXVII/ion:remove-get-variables into master

The GetVariables Trait encouraged cloning rather than using references. Since it is rarely the case that cloning is actually needed, it is better to replace it entirely. Furthermore, most of its usage included matching, so it does not significantly increase general verbosity of the code. To account for the env vars that must be fetched, a new method, get_str, was introduced.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading