remove mutable default; only save create

Author: seeM

fastcore/foundation.py

@@ -249,17 +249,17 @@ def read_config_file(file, **kwargs):
 # %% ../nbs/02_foundation.ipynb 131
 class Config:
     "Reading and writing `ConfigParser` ini files"
-    def __init__(self, cfg_path, cfg_name, create=None, save=True, extra_files=[]):
+    def __init__(self, cfg_path, cfg_name, create=None, save=True, extra_files=None):
         cfg_path = Path(cfg_path).expanduser().absolute()
         self.config_path,self.config_file = cfg_path,cfg_path/cfg_name
         self._cfg = ConfigParser()
         self.d = self._cfg['DEFAULT']
-        found = [Path(o) for o in self._cfg.read(extra_files+[self.config_file])]
+        found = [Path(o) for o in self._cfg.read(L(extra_files)+[self.config_file])]
         if self.config_file not in found and create is not None:
             self._cfg.read_dict({'DEFAULT':create})
             if save:
                 cfg_path.mkdir(exist_ok=True, parents=True)
-                self.save()
+                save_config_file(self.config_file, create)
 
     def __repr__(self): return repr(dict(self._cfg.items('DEFAULT', raw=True)))
     def __setitem__(self,k,v): self.d[k] = str(v)

nbs/02_foundation.ipynb

@@ -2004,17 +2004,17 @@
     "#|export\n",
     "class Config:\n",
     "    \"Reading and writing `ConfigParser` ini files\"\n",
-    "    def __init__(self, cfg_path, cfg_name, create=None, save=True, extra_files=[]):\n",
+    "    def __init__(self, cfg_path, cfg_name, create=None, save=True, extra_files=None):\n",
     "        cfg_path = Path(cfg_path).expanduser().absolute()\n",
     "        self.config_path,self.config_file = cfg_path,cfg_path/cfg_name\n",
     "        self._cfg = ConfigParser()\n",
     "        self.d = self._cfg['DEFAULT']\n",
-    "        found = [Path(o) for o in self._cfg.read(extra_files+[self.config_file])]\n",
+    "        found = [Path(o) for o in self._cfg.read(L(extra_files)+[self.config_file])]\n",
     "        if self.config_file not in found and create is not None:\n",
     "            self._cfg.read_dict({'DEFAULT':create})\n",
     "            if save:\n",
     "                cfg_path.mkdir(exist_ok=True, parents=True)\n",
-    "                self.save()\n",
+    "                save_config_file(self.config_file, create)\n",
     "\n",
     "    def __repr__(self): return repr(dict(self._cfg.items('DEFAULT', raw=True)))\n",
     "    def __setitem__(self,k,v): self.d[k] = str(v)\n",
@@ -2040,7 +2040,7 @@
    "cell_type": "markdown",
    "metadata": {},
    "source": [
-    "Instantiate a `Config` from a ini file at `cfg_path/cfg_name`:"
+    "Instantiate a `Config` from an ini file at `cfg_path/cfg_name`:"
    ]
   },
   {