tree 880106ba779c69e6996617b5c710161b9c7d2eb0
parent 56ece43b53b64c63ae51ec184b76bd5360c28d0b
author James D. Lin <jamesderlin@users.noreply.github.com> 1613156493 -0800
committer GitHub <noreply@github.com> 1613156493 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgJtCNCRBK7hj4Ov3rIwAAdHIIAAKSwkl6WGZbxbX0uKnmzTec
 wRvVx3SiO4xvaEYPPk6IoqfsU6pL8OyGCe8OLzXuGV7ef47cz2q3AqOUQZdORmoA
 n2RDfC432pq76r7MW1tNC8CL1CXjU9np7j93tZztcSKX+Zd/k/sdPcd+n8ptF6Kz
 0SthVfR/o8BhpvAtjVQdJgdLD4Rr9txiumwmtF9SVWshY0Nxn058VR1O4NaNRDad
 nsG6EYNfGnkSactgntkfLWYh1yQtIop35uPcg+fPt9RG/FOmMdPWdGS5BjyRlio3
 11mcbys0+oMD54s7baDoHOdW9Ds1Qpe8YaSb38EdmgQFEm00zru+XZ6y5NUI7lo=
 =qkOg
 -----END PGP SIGNATURE-----
 

Fix ProcessManager signatures to be consistent with LocalProcessManager (#58)

When `LocalProcessManager.run`, `.runSync`, and `.start` were
migrated for null-safety, the base `ProcessManager` interface was not
updated.  This led to an inconsistency where the interface
unnecessarily required non-null versions of `workingDirectory` and
`environment`, which creates extra headache for clients.  This change
should be safe since it makes the base interface less restrictive.

Additionally, `LocalProcessManager` takes a covariant `List<Object>`
where the base interface takes `List<dynamic>`.  I don't see a need
for the interface to take `List<dynamic>` (which could accept `null`)
so change that too.  This change is less safe, although in practice
anything that attempted to pass `null` would already crash if using
the `LocalProcessManager` implementation anyway.

Fixes https://github.com/google/process.dart/issues/56.