Use IEnumerable<string> rather than list string

Author: KevinRansom

src/fsharp/DependencyManager.Integration.fs

@@ -4,6 +4,7 @@
 module internal FSharp.Compiler.DependencyManagerIntegration
 
 open System
+open System.Collections.Generic
 open System.IO
 open System.Reflection
 open FSharp.Compiler.DotNetFrameworkDependencies
@@ -64,8 +65,9 @@ type internal IDependencyManagerProvider =
     abstract Name: string
     abstract Key: string
     abstract ResolveDependencies: scriptDir: string * mainScriptName: string * scriptName: string * scriptExt: string * packageManagerTextLines: string seq * tfm: string -> bool * string list * string list
+
     abstract DependencyAdding: IEvent<string * string>
-    abstract DependencyAdded: IEvent<string * string * string list * string list * string list>
+    abstract DependencyAdded: IEvent<string * string * IEnumerable<string> * IEnumerable<string> * IEnumerable<string>>
     abstract DependencyFailed: IEvent<string * string>
 
 [<RequireQualifiedAccess>]
@@ -79,9 +81,9 @@ type ReflectionDependencyManagerProvider(theType: Type, nameProperty: PropertyIn
     let nameProperty     = nameProperty.GetValue >> string
     let keyProperty      = keyProperty.GetValue >> string
 
-    let dependencyAddingEvent = Event<_>()
-    let dependencyAddedEvent = Event<_>()
-    let dependencyFailedEvent = Event<_>()
+    let dependencyAddingEvent = Event<string * string>()
+    let dependencyAddedEvent = Event<string * string * IEnumerable<string> * IEnumerable<string> * IEnumerable<string>>()
+    let dependencyFailedEvent = Event<string * string>()
 
     static member InstanceMaker (theType: System.Type, outputDir: string option) =
         match ReflectionHelper.getAttributeNamed theType dependencyManagerAttributeName,
@@ -144,7 +146,7 @@ type ReflectionDependencyManagerProvider(theType: Type, nameProperty: PropertyIn
 
             for prLine in packageManagerTextLines do
                 if succeeded then
-                    dependencyAddedEvent.Trigger(key, prLine, references, generatedScripts, additionalIncludeFolders)
+                    dependencyAddedEvent.Trigger(key, prLine, references |> List.toSeq, generatedScripts |> List.toSeq, additionalIncludeFolders |> List.toSeq)
                 else
                     dependencyFailedEvent.Trigger(key, prLine)
 

src/fsharp/DependencyManager.Integration.fsi

@@ -3,14 +3,15 @@
 /// Helper members to integrate DependencyManagers into F# codebase
 module internal FSharp.Compiler.DependencyManagerIntegration
 
+open System.Collections.Generic
 open FSharp.Compiler.Range
 
 type IDependencyManagerProvider =
     abstract Name: string
     abstract Key: string
     abstract ResolveDependencies: scriptDir: string * mainScriptName: string * scriptName: string * scriptExt: string * packageManagerTextLines: string seq * tfm: string -> bool * string list * string list
     abstract DependencyAdding: IEvent<string * string>
-    abstract DependencyAdded: IEvent<string * string * string list * string list * string list>
+    abstract DependencyAdded: IEvent<string * string * IEnumerable<string> * IEnumerable<string> * IEnumerable<string>>
     abstract DependencyFailed: IEvent<string * string>
 
 [<RequireQualifiedAccess>]

src/fsharp/fsi/fsi.fs

@@ -959,7 +959,7 @@ type internal FsiDynamicCompiler
 
     let valueBoundEvent = Control.Event<_>()
     let dependencyAddingEvent = Control.Event<string * string>()
-    let dependencyAddedEvent = Control.Event<string * string * string list * string list * string list>()
+    let dependencyAddedEvent = Control.Event<string * string * IEnumerable<string> * IEnumerable<string> * IEnumerable<string>>()
     let dependencyFailedEvent = Control.Event<string * string>()
 
     let mutable fragmentId = 0

src/fsharp/fsi/fsi.fsi

@@ -3,6 +3,7 @@
 
 module public FSharp.Compiler.Interactive.Shell
 
+open System.Collections.Generic
 open System.IO
 open System.Threading
 open FSharp.Compiler
@@ -240,7 +241,7 @@ type FsiEvaluationSession =
 
     [<CLIEvent>]
     /// Event fires at the successful completion of adding a dependency via the dependency manager.
-    member DependencyAdded: IEvent<string * string * string list * string list * string list>
+    member DependencyAdded: IEvent<string * string * IEnumerable<string> * IEnumerable<string> * IEnumerable<string>>
 
     [<CLIEvent>]
     /// Event fires at the failure to adding a dependency via the dependency manager.