Oh, so blank lines should have the same indentation as whatever function
they're nested in?


On Wed, Jun 4, 2014 at 2:04 PM, Marianne Corvellec <notifications@github.com
> wrote:

> In R/plotly.R:
>
> >      # Parse status
> >      if (resp_header$status != "200") {
> >        cat(resp_header$statusMsg)
> >        stop(resp_header$status)
> >      }
> > -
>
> Yes. There are two tabs there: We are inside function get_figure() which
> itself is in function plotly(). Arguably you could remove the line all
> together. But if you keep this line skip, it should be indented with two
> tabs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ropensci/plotly/pull/46/files#r13401081>.
>