Extract into const

[yoav-lavi]

Hey, I think a useful action for &'static str would be "Extract into const":

let opt_env_var = std::env::var("SOME_ENV_VAR").ok();

➜ Extract into const

const SOME_ENV_VAR: &str = "SOME_ENV_VAR";

let opt_env_var = std::env::var(SOME_ENV_VAR).ok();

Suggested behavior:

const {SCREAMING_SNAKE_CASE_STR_CONTENT}: &str = {str_content};

/* previous location ➜ */ {SCREAMING_SNAKE_CASE_STR_CONTENT}

• The name could either be the &'static str content or a generic placeholder. The content is beneficial since it may reduce the amount of times the const needs to be renamed, but it's less likely to be correct for longer content.
• Not 100% sure if it should be moved to the root scope or to the top of the current scope
• Could be extended to all literals with different rules for naming

Thanks for your consideration!

[yoav-lavi]

Looks like this may be a duplicate of #7692, currently leaving this open since it has some extra suggestions for behavior but can be closed

[lnicola]

Closing in favor of #18679.