FYI just made this change to rmarkdown.


On Thu, Apr 24, 2014 at 8:02 PM, JJ Allaire <jj@rstudio.com> wrote:

> Agreed that aggregating isn't useful (we should also eliminate this from
> rmarkdown)
>
>
> On Thursday, April 24, 2014, Jonathan <notifications@github.com> wrote:
>
>> In R/html-deps.R:
>>
>> > +  }
>> > +
>> > +  return(dependency)
>> > +}
>> > +
>> > +# Given a list of dependencies, choose the latest versions and return them as a
>> > +# named list in the correct order.
>> > +getNewestDeps <- function(dependencies) {
>> > +  result <- list()
>> > +  for (dep in dependencies) {
>> > +    if (!is.null(dep)) {
>> > +      other <- result[[dep$name]]
>> > +      if (is.null(other) || compareVersion(dep$version, other$version) > 0) {
>> > +        # Note that if the dep was already in the result list, then this
>> > +        # assignment preserves its position in the list
>> > +        result[[dep$name]] <- dep
>>
>> In the rmarkdown version of this algorithm (
>> https://github.com/rstudio/rmarkdown/blob/master/R/html_dependencies.R#L128-L131),
>> the meta and head information from the dependencies are aggregated
>> rather than replaced--should we do that here?
>>
>> --
>> Reply to this email directly or view it on GitHub<https://github.com/rstudio/shiny/pull/442/files#r11976874>
>> .
>>
>