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.

Sign up for GitHub

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

Jump to bottom

Planned removal of report_todo #5101

Closed
calebcartwright opened this issue Nov 22, 2021 · 2 comments · Fixed by #5357
Closed

Planned removal of report_todo #5101

calebcartwright opened this issue Nov 22, 2021 · 2 comments · Fixed by #5357
Assignees
@calebcartwright

Comments

@calebcartwright
Copy link
Member

The purpose of this issue is to both communicate our intent to drop the unstable option report_todo, as well as provide a forum for questions to be asked and/or concerns to be raised.

We do not yet have an exact deprecation date planned, but it will most likely occur in early 2022.

While we understand the utility this option provides, we've decided that rustfmt simply is not the right place/tool to provide this capability, and it's not an option that makes sense for us to try to stabilize and maintain/support long term.

If anyone is aware of any other community maintained tools/crates for detecting "TODO" strings in comments please feel free to post here for broader awareness.
@calebcartwright calebcartwright pinned this issue Nov 22, 2021
@calebcartwright calebcartwright self-assigned this Nov 22, 2021
@calebcartwright calebcartwright mentioned this issue Nov 22, 2021
Closed
@scott-ainsworth
Copy link

Just for clarity, and because report_todo and report_fixme are practicality synonymous to many people, is just report_todo being depreciated? Tx

@calebcartwright
Copy link
Member Author

Thanks for the question, and yes report_fixme is being deprecated as well - #5102

@calebcartwright calebcartwright mentioned this issue Dec 8, 2021
Closed
@ytmimi ytmimi mentioned this issue Mar 31, 2022
Merged
This was referenced May 30, 2022
Closed
Merged
@ascjones ascjones mentioned this issue Jul 5, 2022
Merged
@VisualEhrmanntraut VisualEhrmanntraut mentioned this issue Aug 30, 2022
Closed
@calebcartwright calebcartwright unpinned this issue Dec 20, 2022
@apollo-bobby apollo-bobby mentioned this issue Jan 10, 2023
Open
@SWvheerden SWvheerden mentioned this issue Feb 1, 2023
Merged
stringhandler pushed a commit to tari-project/tari that referenced this issue Feb 1, 2023
@SWvheerden
chore: fix cargo fmt (#5159)
603cb1f 
Description
---
Removes unused cargo fmt options

Motivation and Context
---
Since [[1.5.0] 2022-06-14](https://github.com/rust-lang/rustfmt/blob/1d8491b120223272b13451fc81265aa64f7f4d5b/CHANGELOG.md#150-2022-06-14) the options `report_todo` and `report_fixme`  have been removed,  [5101](rust-lang/rustfmt#5101) and [5102](rust-lang/rustfmt#5102)

How Has This Been Tested?
---
Manual 
Cargo fmt does not complain about: 
```
Warning: Unknown configuration option `report_fixme`
Warning: Unknown configuration option `report_todo`
```
@tareknaser tareknaser mentioned this issue Feb 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@calebcartwright calebcartwright

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove some unstable config options that've been deprecated calebcartwright/rustfmt
2 participants
@scott-ainsworth @calebcartwright