refactor: remove skipAutoReload option

I don't think it's necessary - doesn't feel like it makes sense to set
those types of flags in module configuration, rather than command
configuration.

src/commands/auto-reload.ts

@@ -18,14 +18,11 @@ import { registerCleanupFunction, sleep } from "../util"
 export type AutoReloadDependants = { [key: string]: Set<Module> }
 
 async function registerAutoReloadWatches(ctx: PluginContext): Promise<FSWatcher | null> {
-  const allModules = values(await ctx.getModules())
-  const modules = allModules.filter((m) => !m.skipAutoReload)
+  const modules = values(await ctx.getModules())
 
   if (modules.length === 0) {
-    if (allModules.length === 0) {
+    if (modules.length === 0) {
       ctx.log.info({ msg: "No modules found in project." })
-    } else {
-      ctx.log.info({ msg: "All modules in project have skipAutoReload = true." })
     }
     ctx.log.info({ msg: "Aborting..." })
     return null
@@ -50,7 +47,7 @@ export async function computeAutoReloadDependants(modules: Module[]):
 
   for (const module of modules) {
     const deps = await module.getBuildDependencies()
-    for (const dep of deps.filter(d => !d.skipAutoReload)) {
+    for (const dep of deps) {
       dependants[dep.name] = (dependants[dep.name] || new Set()).add(module)
     }
   }

src/types/module.ts

@@ -61,7 +61,6 @@ export interface ModuleConfig<T extends ServiceConfig = ServiceConfig> {
   name: string
   path: string
   services: { [name: string]: T }
-  skipAutoReload?: boolean
   test: TestConfig
   type: string
   variables: PrimitiveMap
@@ -72,7 +71,6 @@ export class Module<T extends ModuleConfig = ModuleConfig> {
   public type: string
   public path: string
   public services: T["services"]
-  public skipAutoReload: boolean
 
   private _buildDependencies: Module[]
 
@@ -83,7 +81,6 @@ export class Module<T extends ModuleConfig = ModuleConfig> {
     this.type = config.type
     this.path = config.path
     this.services = config.services
-    this.skipAutoReload = config.skipAutoReload || false
   }
 
   @Memoize()

test/src/commands/auto-reload.ts

@@ -82,7 +82,6 @@ async function watch(
 
 // async function testAutoReload(ctx: GardenContext, moduleName: string) {
 //   const modules = values(await ctx.getModules())
-//     .filter(m => !m.skipAutoReload)
 //   const autoReloadDependants = await computeAutoReloadDependants(modules)
 //   const entryModule = modules.find(m => m.name === moduleName) as Module
 //