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

Warning flags #20

Closed
hollance opened this issue Nov 8, 2013 · 5 comments
Closed

Warning flags #20

hollance opened this issue Nov 8, 2013 · 5 comments

Comments

@hollance
Copy link
Member

hollance commented Nov 8, 2013

What are good warning flags to enable by default?

I prefer -Wall -Wextra -Wno-unused-parameter. It might be good to have these as suggested settings.

@rwenderlich
Copy link
Member

Good idea.

But how should we handle this in tutorials? Should we direct the reader to add this flag during project setup, or should we just do it while developing the project behind the scenes?

@gregheo
Copy link
Contributor

gregheo commented Nov 10, 2013

We can mention it as the recommended setting as a tip to readers, and ask authors to add it to sample/final projects attached to tutorials. I wouldn't want to mention it every time if a tutorial is a "start a project from scratch" type.

@ghost
Copy link

ghost commented Nov 12, 2013

I don't believe warning flags belong in a style guide, as they don't pertain to style and have no direct impact on formatting. IMO these belong more in a 'Best Practices' type guide.

@ndubbs
Copy link
Contributor

ndubbs commented Nov 17, 2013

I agree with @micpringle regarding 'Best Practices'.

+1 for not including this in the style guide.

@rwenderlich
Copy link
Member

Good point, yeah it does make sense to put this guideline in a best practices guide instead of a style guide. Maybe we can just put it on the tutorial team guide somewhere.

@ndubbs ndubbs closed this as completed Dec 19, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants