Remove unecessary .to_string #106

Merged

notplants merged 1 commits from to-string into main

2022-05-11 09:48:16 +00:00

notplants commented

2022-05-11 09:26:24 +00:00

Owner

was able to use

let pc_defaults: HashMap<String, String> = peach_config_defaults
        .iter()
        .map(|(key, val)| (key.to_string(), val.to_string()))
        .collect();

to convert a HashMap<&str, &str> into a HashMap<String, String>,

so that we don't need to write .to_string() on every single string in the defaults.

was able to use ``` let pc_defaults: HashMap<String, String> = peach_config_defaults .iter() .map(|(key, val)| (key.to_string(), val.to_string())) .collect(); ``` to convert a HashMap<&str, &str> into a HashMap<String, String>, so that we don't need to write .to_string() on every single string in the defaults.

notplants added 1 commit 2022-05-11 09:26:25 +00:00

continuous-integration/drone/pr Build is passing Details

b84e470a42 Remove unecessary .to_string

notplants requested review from glyph 2022-05-11 09:26:31 +00:00

glyph approved these changes 2022-05-11 09:39:45 +00:00

glyph left a comment

Owner

👍 🟢

👍 🟢

notplants merged commit bab33b602a into main

2022-05-11 09:48:16 +00:00

notplants deleted branch to-string

2022-05-11 09:48:17 +00:00

notplants referenced this issue from a commit

2022-05-11 09:48:19 +00:00

Merge pull request 'Remove unecessary .to_string' (#106) from to-string into main

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: PeachCloud/peach-workspace#106

No description provided.