Windows CI support

.github/workflows/build.yml

@@ -9,7 +9,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, macos-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
         python-version: [3.6, 3.7, 3.8, 3.9]
     steps:
     - uses: actions/checkout@v1
@@ -33,7 +33,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, macos-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
         python-version: [3.6, 3.7, 3.8, 3.9]
     steps:
     - uses: actions/checkout@v1

libcst/codemod/_cli.py

@@ -16,7 +16,7 @@
 import traceback
 from dataclasses import dataclass, replace
 from multiprocessing import Pool, cpu_count
-from pathlib import Path
+from pathlib import Path, PurePath
 from typing import Any, AnyStr, Dict, List, Optional, Sequence, Union, cast
 
 from libcst import PartialParserConfig, parse_module
@@ -192,26 +192,21 @@ def _calculate_module(repo_root: Optional[str], filename: str) -> Optional[str]:
         # We don't have a repo root, so this is impossible to calculate.
         return None
 
-    # Make sure the absolute path for the root ends in a separator.
-    if repo_root[-1] != os.path.sep:
-        repo_root = repo_root + os.path.sep
-
-    if not filename.startswith(repo_root):
+    try:
+        relative_filename = PurePath(filename).relative_to(repo_root)
+    except ValueError:
         # This file seems to be out of the repo root.
         return None
 
-    # Get the relative path, get rid of any special cases and extensions.
-    relative_filename = filename[len(repo_root) :]
-    for ending in [
-        f"{os.path.sep}__init__.py",
-        f"{os.path.sep}__main__.py",
-        ".py",
-    ]:
-        if relative_filename.endswith(ending):
-            relative_filename = relative_filename[: -len(ending)]
-
-    # Now, convert all line separators to dots to represent the python module.
-    return relative_filename.replace(os.path.sep, ".")
+    # get rid of extension
+    relative_filename = relative_filename.with_suffix("")
+
+    # get rid of any special cases
+    if relative_filename.stem in ["__init__", "__main__"]:
+        relative_filename = relative_filename.parent
+
+    # Now, convert to dots to represent the python module.
+    return ".".join(relative_filename.parts)
 
 
 @dataclass(frozen=True)

libcst/codemod/tests/test_cli.py

@@ -37,6 +37,27 @@ class TestPackageCalculation(UnitTest):
                 "/home/username/root/some/dir/__main__.py",
                 "some.dir",
             ),
+            # some windows tests
+            (
+                "c:/Program Files/",
+                "d:/Program Files/some/dir/file.py",
+                None,
+            ),
+            (
+                "c:/Program Files/other/",
+                "c:/Program Files/some/dir/file.py",
+                None,
+            ),
+            (
+                "c:/Program Files/",
+                "c:/Program Files/some/dir/file.py",
+                "some.dir.file",
+            ),
+            (
+                "c:/Program Files/",
+                "c:/Program Files/some/dir/__main__.py",
+                "some.dir",
+            ),
         ),
     )
     def test_calculate_module(