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

Add a flag that enables all linting #18559

Closed
szymon-rd opened this issue Sep 15, 2023 · 4 comments · Fixed by #20577
Closed

Add a flag that enables all linting #18559

szymon-rd opened this issue Sep 15, 2023 · 4 comments · Fixed by #20577
Assignees
@Linyxus
Labels
area:linting Linting warnings enabled with -W or -Xlint area:settings Issues tied to command line options & settings. itype:enhancement

Comments

@szymon-rd
Copy link
Contributor

Having a flag enabling all the linting options at once would be helpful. More and more linting is being added, and writing all the flags can make the compile command long. It is also a standard user path to enable all the linting options.
@szymon-rd szymon-rd added itype:enhancement Spree Suitable for a future Spree area:linting Linting warnings enabled with -W or -Xlint labels Sep 15, 2023
@som-snytt
Copy link
Contributor

The topic of research is what to name it.

I previously tried legacy -Xlint #14753

Conforming -Wlint isn't pronounceable.

Maybe there is a linguist at EPFL.

@SethTisue
Copy link
Member

fwiw, gcc uses -Wall (and has for donkey's years)

@som-snytt
Copy link
Contributor

I happen to be reading Moby-Dick, -Wal.

@szymon-rd szymon-rd mentioned this issue Oct 2, 2023
Closed
@mbovel mbovel removed the Spree Suitable for a future Spree label Jan 14, 2024
@szymon-rd szymon-rd mentioned this issue Jan 31, 2024
Closed
@Gedochao Gedochao added the area:settings Issues tied to command line options & settings. label Apr 4, 2024
@Linyxus Linyxus mentioned this issue Apr 22, 2024
Draft
@Linyxus Linyxus mentioned this issue Jun 10, 2024
Closed
@Gedochao Gedochao mentioned this issue Jun 27, 2024
Merged
@Linyxus
Copy link
Contributor

Linyxus commented Aug 22, 2024

closed by #20577

@Linyxus Linyxus closed this as completed Aug 22, 2024
@Gedochao Gedochao linked a pull request Aug 22, 2024 that will close this issue
Add the -Wall option that enables all warnings (Plan B) #20577
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Linyxus Linyxus

Labels
area:linting Linting warnings enabled with -W or -Xlint area:settings Issues tied to command line options & settings. itype:enhancement
Projects
None yet
Milestone
No milestone
6 participants
@SethTisue @som-snytt @mbovel @szymon-rd @Linyxus @Gedochao