Skip to content
New issue

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

Remove unstable option license_template_path #5246

Closed
wants to merge 1 commit into from

Conversation

ytmimi
Copy link
Contributor

@ytmimi ytmimi commented Mar 1, 2022

Closes #5103

It was decided that rustfmt is not the right tool to provide this feature.

Closes 5103

It was decided that rustfmt is not the right tool to provide this
feature.
@calebcartwright
Copy link
Member

Thanks for this, though I'm going to close as I'm still holding off on the removal for a bit longer. I'll drop all three in unison when it's time

@ytmimi
Copy link
Contributor Author

ytmimi commented Mar 1, 2022

No worries, I thought I'd get the ball rolling on the planned removals. I was going to follow up with PRs to remove the other options, but totally understand if you want to do that all in one go.

FintanH added a commit to radicle-dev/radicle-surf that referenced this pull request Jul 11, 2022

Verified

This commit was signed with the committer’s verified signature. The key has expired.
FintanH Fintan Halpenny
Update to rust-1.62.0[0]. This requires converting `push_str` +
`format!` calls into a `write!` call.

Remove `license_template_path` since it is deprecated[1].

[0]: https://blog.rust-lang.org/2022/06/30/Rust-1.62.0.html
[1]: rust-lang/rustfmt#5246

Signed-off-by: Fintan Halpenny <fintan.halpenny@gmail.com>
@ytmimi ytmimi deleted the remove_license_check branch August 7, 2022 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Planned removal of license_template_path
2 participants