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

Make wrap=FALSE by default #178

Closed
yihui opened this issue Jan 27, 2014 · 3 comments
Closed

Make wrap=FALSE by default #178

yihui opened this issue Jan 27, 2014 · 3 comments

Comments

@yihui
Copy link
Contributor

@yihui yihui commented Jan 27, 2014

so that we can retire Roxygen: list(wrap = FALSE) in DESCRIPTION. Is this going to happen some day?

Or at least make it configurable as an argument in roxygenize().

@hadley
Copy link
Member

@hadley hadley commented Jan 27, 2014

Yes, the plan is to do this gradually.

yihui added a commit to yihui/roxygen2 that referenced this issue Jan 28, 2014
yihui added a commit to yihui/roxygen2 that referenced this issue Jan 28, 2014
@yihui
Copy link
Contributor Author

@yihui yihui commented Jan 28, 2014

I have sent a PR to expedite this :)

@hadley
Copy link
Member

@hadley hadley commented Feb 5, 2014

I now think we don't need to be so cautious - I think we can just print a message describing what is happening (i.e. wrap value taken from DESCRIPTION Roxygen: list(wrap = TRUE)) and then change the default to FALSE.

yihui added a commit to yihui/roxygen2 that referenced this issue Feb 6, 2014
yihui added a commit to yihui/roxygen2 that referenced this issue Feb 6, 2014
@hadley hadley closed this in 3cd0e0a Feb 7, 2014
hadley added a commit that referenced this issue Feb 7, 2014
Default to wrap = FALSE with a message. Closes #178
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants