fixes #1621

Author: jph00

nbdev/config.py

@@ -11,18 +11,16 @@
            'update_llms_txt', 'import_obj', 'write_cells']
 
 # %% ../nbs/api/01_config.ipynb #6fd14ecd
-from datetime import datetime
 from fastcore.docments import *
 from fastcore.utils import *
 from fastcore.meta import *
 from fastcore.script import *
 from fastcore.style import *
 from fastcore.xdg import *
 
-import ast,importlib,re,warnings
+import ast,importlib,re
 from IPython.display import Markdown
 from fastcore.nbio import read_nb,NbCell
-from urllib.error import HTTPError
 
 try: import tomllib
 except ImportError: import tomli as tomllib
@@ -42,6 +40,7 @@ def _get_info(owner, repo, default_branch='main', default_kw='nbdev'):
     from ghapi.all import call_gh, APIError
     try: r = call_gh('repos.get', owner=owner, repo=repo)
     except APIError:
+        # chkstyle: ignore-node
         msg= [f"""Could not access repo: {owner}/{repo} to find your default branch - `{default_branch}` assumed.
 Edit `pyproject.toml` if this is incorrect.
 In the future, you can allow nbdev to see private repos by setting the environment variable GITHUB_TOKEN as described here:
@@ -205,8 +204,7 @@ def __init__(self, d, proj, cfg_file):
         if 'lib_path' not in self: self['lib_path'] = self.lib_name.replace('-', '_')
 
     @property
-    def version(self):
-        return read_version(self.config_path / self['lib_path']) or '0.0.1'
+    def version(self): return read_version(self.config_path / self['lib_path']) or '0.0.1'
 
     @property
     def d(self): return {k:v for k,v in super().items()}
@@ -253,7 +251,7 @@ def is_nbdev(path=None): return _find_nbdev_pyproject(path) is not None
 def create_output(txt, mime):
     "Add a cell output containing `txt` of the `mime` text MIME sub-type"
     return [dict(data={f"text/{mime}": str(txt).splitlines(True)},
-                 execution_count=1, metadata={}, output_type="execute_result")]
+        execution_count=1, metadata={}, output_type="execute_result")]
 
 # %% ../nbs/api/01_config.ipynb #5a4d8e52
 def show_src(src, lang='python'): return Markdown(f'```{lang}\n{src}\n```')
@@ -318,12 +316,13 @@ def set_version(path, version):
 
 # %% ../nbs/api/01_config.ipynb #d00889e5
 def bump_version(v, part=2, unbump=False):
-    "Bump semver string `v` at index `part` (0=major, 1=minor, 2=patch)"
+    "Bump semver part `part` (0=major, 1=minor, 2=patch), counted from the right"
     parts = (v or '0.0.0').split('.')
     parts += ['0'] * (3 - len(parts))
-    parts[part] = str(int(parts[part]) + (-1 if unbump else 1))
-    for i in range(part+1, 3): parts[i] = '0'
-    return '.'.join(parts[:3])
+    idx = len(parts) - 3 + part
+    parts[idx] = str(int(parts[idx]) + (-1 if unbump else 1))
+    for i in range(idx+1, len(parts)): parts[i] = '0'
+    return '.'.join(parts)
 
 # %% ../nbs/api/01_config.ipynb #e32583e6
 def update_version(path=None):

nbdev/test.py

@@ -108,8 +108,8 @@ def nbdev_test(
     if len(files)==0: return print('No files were eligible for testing')
 
     if n_workers is None: n_workers = 0 if len(files)==1 else min(num_cpus(), 8)
-    if IN_NOTEBOOK: kw = {'method':'spawn'} if os.name=='nt' else {'method':'forkserver'}
-    else: kw = {'method':'forkserver'} if sys.platform=='darwin' else {}
+    if IN_NOTEBOOK: kw = {'method':'spawn'} if os.name=='nt' or sys.platform=='darwin' else {'method':'forkserver'}
+    else: kw = {'method':'spawn'} if sys.platform=='darwin' else {}
     wd_pth = cfg.nbs_path
     with working_directory(wd_pth if (wd_pth and wd_pth.exists()) else os.getcwd()):
         results = parallel(test_nb, files, skip_flags=skip_flags, force_flags=force_flags, n_workers=n_workers,

nbs/api/01_config.ipynb

@@ -30,18 +30,16 @@
    "outputs": [],
    "source": [
     "#| export\n",
-    "from datetime import datetime\n",
     "from fastcore.docments import *\n",
     "from fastcore.utils import *\n",
     "from fastcore.meta import *\n",
     "from fastcore.script import *\n",
     "from fastcore.style import *\n",
     "from fastcore.xdg import *\n",
     "\n",
-    "import ast,importlib,re,warnings\n",
+    "import ast,importlib,re\n",
     "from IPython.display import Markdown\n",
     "from fastcore.nbio import read_nb,NbCell\n",
-    "from urllib.error import HTTPError\n",
     "\n",
     "try: import tomllib\n",
     "except ImportError: import tomli as tomllib"
@@ -138,6 +136,7 @@
     "    from ghapi.all import call_gh, APIError\n",
     "    try: r = call_gh('repos.get', owner=owner, repo=repo)\n",
     "    except APIError:\n",
+    "        # chkstyle: ignore-node\n",
     "        msg= [f\"\"\"Could not access repo: {owner}/{repo} to find your default branch - `{default_branch}` assumed.\n",
     "Edit `pyproject.toml` if this is incorrect.\n",
     "In the future, you can allow nbdev to see private repos by setting the environment variable GITHUB_TOKEN as described here:\n",
@@ -398,8 +397,7 @@
     "        if 'lib_path' not in self: self['lib_path'] = self.lib_name.replace('-', '_')\n",
     "\n",
     "    @property\n",
-    "    def version(self):\n",
-    "        return read_version(self.config_path / self['lib_path']) or '0.0.1'\n",
+    "    def version(self): return read_version(self.config_path / self['lib_path']) or '0.0.1'\n",
     "\n",
     "    @property\n",
     "    def d(self): return {k:v for k,v in super().items()}\n",
@@ -604,7 +602,7 @@
     "def create_output(txt, mime):\n",
     "    \"Add a cell output containing `txt` of the `mime` text MIME sub-type\"\n",
     "    return [dict(data={f\"text/{mime}\": str(txt).splitlines(True)},\n",
-    "                 execution_count=1, metadata={}, output_type=\"execute_result\")]"
+    "        execution_count=1, metadata={}, output_type=\"execute_result\")]"
    ]
   },
   {
@@ -810,12 +808,13 @@
    "source": [
     "#| export\n",
     "def bump_version(v, part=2, unbump=False):\n",
-    "    \"Bump semver string `v` at index `part` (0=major, 1=minor, 2=patch)\"\n",
+    "    \"Bump semver part `part` (0=major, 1=minor, 2=patch), counted from the right\"\n",
     "    parts = (v or '0.0.0').split('.')\n",
     "    parts += ['0'] * (3 - len(parts))\n",
-    "    parts[part] = str(int(parts[part]) + (-1 if unbump else 1))\n",
-    "    for i in range(part+1, 3): parts[i] = '0'\n",
-    "    return '.'.join(parts[:3])"
+    "    idx = len(parts) - 3 + part\n",
+    "    parts[idx] = str(int(parts[idx]) + (-1 if unbump else 1))\n",
+    "    for i in range(idx+1, len(parts)): parts[i] = '0'\n",
+    "    return '.'.join(parts)"
    ]
   },
   {
@@ -842,7 +841,10 @@
     "test_eq(bump_version('1.2.3'), '1.2.4')\n",
     "test_eq(bump_version('1.2.3', part=1), '1.3.0')\n",
     "test_eq(bump_version('1.2.3', part=0), '2.0.0')\n",
-    "test_eq(bump_version('1.2.3', part=2, unbump=True), '1.2.2')"
+    "test_eq(bump_version('1.2.3', part=2, unbump=True), '1.2.2')\n",
+    "test_eq(bump_version('2026.05.27.2'), '2026.05.27.3')\n",
+    "test_eq(bump_version('2026.05.27.2', part=1), '2026.05.28.0')\n",
+    "test_eq(bump_version('2026.05.27.2', part=0), '2026.6.0.0')"
    ]
   },
   {
@@ -1038,7 +1040,8 @@
     "from fastcore.nbio import new_nb, mk_cell, write_nb\n",
     "with tempfile.TemporaryDirectory() as d:\n",
     "    d = Path(d)\n",
-    "    p = d/'pkg'; p.mkdir()\n",
+    "    p = d/'pkg'\n",
+    "    p.mkdir()\n",
     "    (p/'a.py').write_text('\"\"\"A module\\nspanning lines\\n\\nMore about a.\\n\\nDocs: http://x\"\"\"\\n')\n",
     "    (p/'b.py').write_text('\"\"\"B summary\\n\\nDocs: http://y\"\"\"\\n')\n",
     "    (p/'_c.py').write_text('\"\"\"Private\\n\\nwith\\nmany\\nlines\"\"\"\\n')\n",
@@ -1054,14 +1057,15 @@
     "    update_init_mdoc(p, desc='Test pkg')\n",
     "    assert '- `pkg.b`: B summary' in (p/'__init__.py').read_text()\n",
     "    idx = new_nb([mk_cell('# Pkg\\n> tagline', 'markdown'), mk_cell('#| export\\nIntro para.', 'markdown'),\n",
-    "                  mk_cell('#| exportd\\nprint(1)'), mk_cell('Untagged prose.', 'markdown')])\n",
+    "        mk_cell('#| exportd\\nprint(1)'), mk_cell('Untagged prose.', 'markdown')])\n",
     "    write_nb(idx, d/'index.ipynb')\n",
     "    m = pkg_mdoc(p, desc='Test pkg', index_nb=d/'index.ipynb')\n",
     "    assert m.startswith('Test pkg\\n\\nIntro para.\\n\\n```python\\nprint(1)\\n```\\n\\nModules:\\n\\n- `pkg.a`')\n",
     "    assert 'Untagged' not in m and 'tagline' not in m\n",
-    "    q = d/'empty_pkg'; q.mkdir()\n",
+    "    q = d/'empty_pkg'\n",
+    "    q.mkdir()\n",
     "    test_eq(pkg_mdoc(q, desc='D', index_nb=d/'index.ipynb'),\n",
-    "            'D\\n\\nIntro para.\\n\\n```python\\nprint(1)\\n```')  # intro alone still makes a docstring"
+    "        'D\\n\\nIntro para.\\n\\n```python\\nprint(1)\\n```')  # intro alone still makes a docstring"
    ]
   },
   {
@@ -1074,13 +1078,15 @@
     "#| hide\n",
     "with tempfile.TemporaryDirectory() as d:\n",
     "    d = Path(d)\n",
-    "    p = d/'pkg'; p.mkdir()\n",
+    "    p = d/'pkg'\n",
+    "    p.mkdir()\n",
     "    (p/'a.py').write_text('\"\"\"A module\\n\\nMore about a.\\n\\nDocs: http://x/a.html.md\"\"\"\\n')\n",
     "    (p/'b.py').write_text('\"\"\"B, no url\\n\\nSubstantive tho.\\n\\nno docs line\"\"\"\\n')\n",
     "    (p/'d.py').write_text('\"\"\"D two urls\\n\\nBody.\\n\\nDocs: http://x/1.md and http://x/2.md\"\"\"\\n')\n",
     "    idx = new_nb([mk_cell('# Pkg\\n> tagline', 'markdown'), mk_cell('#| export\\nIntro para.', 'markdown')])\n",
     "    write_nb(idx, d/'index.ipynb')\n",
     "    s = pkg_llms(p, desc='Test pkg', index_nb=d/'index.ipynb')\n",
+    "    # chkstyle: ignore-node\n",
     "    test_eq(s, '''# pkg\n",
     "\n",
     "> Test pkg\n",
@@ -1120,8 +1126,7 @@
     "    (d/_pyproj).write_text(cfg_text)\n",
     "    (d/'testpkg').mkdir()\n",
     "\n",
-    "    with working_directory(d):\n",
-    "        update_proj(d)\n",
+    "    with working_directory(d): update_proj(d)\n",
     "\n",
     "    result = (d/_pyproj).read_text()\n",
     "    assert 'name = \"testpkg\"' in result"

nbs/api/12_test.ipynb

@@ -236,8 +236,8 @@
     "    if len(files)==0: return print('No files were eligible for testing')\n",
     "\n",
     "    if n_workers is None: n_workers = 0 if len(files)==1 else min(num_cpus(), 8)\n",
-    "    if IN_NOTEBOOK: kw = {'method':'spawn'} if os.name=='nt' else {'method':'forkserver'}\n",
-    "    else: kw = {'method':'forkserver'} if sys.platform=='darwin' else {}\n",
+    "    if IN_NOTEBOOK: kw = {'method':'spawn'} if os.name=='nt' or sys.platform=='darwin' else {'method':'forkserver'}\n",
+    "    else: kw = {'method':'spawn'} if sys.platform=='darwin' else {}\n",
     "    wd_pth = cfg.nbs_path\n",
     "    with working_directory(wd_pth if (wd_pth and wd_pth.exists()) else os.getcwd()):\n",
     "        results = parallel(test_nb, files, skip_flags=skip_flags, force_flags=force_flags, n_workers=n_workers,\n",