Interpret --skip arguments as relative paths

The --skip arguments are interpreted as paths relative to the
directories to check (the "file" arguments passed to codespell),
in addition to being interpreted "as is".

The benefit is that "codespell ." and "codespell $PWD" behave
the same, you just pass relative paths to --skip. No more adding
"./" to directory patterns.

README.rst

@@ -83,14 +83,14 @@ Comma-separated list of files to skip. It accepts globs as well.  Examples:
 
 * to skip .eps & .txt files, invoke ``codespell --skip="*.eps,*.txt"``
 
-* to skip directories, invoke ``codespell --skip="./src/3rd-Party,./src/Test"``
+* to skip directories, invoke ``codespell --skip="src/3rd-Party,src/Test"``
 
 
 Useful commands:
 
 .. code-block:: sh
 
-    codespell -d -q 3 --skip="*.po,*.ts,./src/3rdParty,./src/Test"
+    codespell -d -q 3 --skip="*.po,*.ts,src/3rdParty,src/Test"
 
 List all typos found except translation files and some directories.
 Display them without terminal colors and with a quiet level of 3.
@@ -129,7 +129,7 @@ be specified in this file (without the preceding dashes), for example:
 .. code-block:: ini
 
     [codespell]
-    skip = *.po,*.ts,./src/3rdParty,./src/Test
+    skip = *.po,*.ts,src/3rdParty,src/Test
     count =
     quiet-level = 3
 
@@ -149,7 +149,7 @@ These are both equivalent to running:
 
 .. code-block:: sh
 
-    codespell --quiet-level 3 --count --skip "*.po,*.ts,./src/3rdParty,./src/Test"
+    codespell --quiet-level 3 --count --skip "*.po,*.ts,src/3rdParty,src/Test"
 
 If several config files are present, they are read in the following order:
 

codespell_lib/_codespell.py

@@ -602,6 +602,10 @@ def parse_options(
         # Re-parse command line options to override config.
         options = parser.parse_args(list(args), namespace=options)
 
+    # Add a phony option than can be changed into a real option when we
+    # are ready to deprecate old patterns (./foo/bar instead of foo/bar)
+    options.deprecated = True
+
     if not options.files:
         options.files.append(".")
 
@@ -1141,28 +1145,44 @@ def main(*args: str) -> int:
 
     bad_count = 0
     for filename in options.files:
-        # ignore hidden files
+        # ignore hidden file or directory
         if is_hidden(filename, options.check_hidden):
             continue
 
         if os.path.isdir(filename):
             for root, dirs, files in os.walk(filename):
-                if glob_match.match(root):  # skip (absolute) directories
+                rel_root = os.path.relpath(root, filename)
+                # skip matching relative and "as is" directory paths
+                if (
+                    glob_match.match(rel_root)
+                    or glob_match.match(root) and options.deprecated
+                ):
                     del dirs[:]
                     continue
-                if is_hidden(root, options.check_hidden):  # dir itself hidden
+
+                # ignore hidden directory
+                if is_hidden(root, options.check_hidden):
                     continue
+
+                # files
                 for file_ in files:
-                    # ignore hidden files in directories
+                    # ignore hidden files
                     if is_hidden(file_, options.check_hidden):
                         continue
-                    if glob_match.match(file_):  # skip files
+                    # skip base file name
+                    if glob_match.match(file_):
                         continue
-                    fname = os.path.join(root, file_)
-                    if glob_match.match(fname):  # skip paths
+                    abs_path = os.path.join(root, file_)
+                    rel_path = os.path.join(rel_root, file_)
+                    # skip matching relative and "as is" file paths
+                    if (
+                        glob_match.match(rel_path)
+                        or glob_match.match(abs_path) and options.deprecated
+                    ):
                         continue
+
                     bad_count += parse_file(
-                        fname,
+                        abs_path,
                         colors,
                         summary,
                         misspellings,
@@ -1176,10 +1196,10 @@ def main(*args: str) -> int:
                         options,
                     )
 
-                # skip (relative) directories
+                # skip base directory name
                 dirs[:] = [dir_ for dir_ in dirs if not glob_match.match(dir_)]
 
-        elif not glob_match.match(filename):  # skip files
+        elif not glob_match.match(filename):  # skip file
             bad_count += parse_file(
                 filename,
                 colors,

codespell_lib/tests/test_basic.py

@@ -384,6 +384,7 @@ def test_encoding(
     assert "WARNING: Binary file" in stderr
 
 
+<<<<<<< HEAD
 def test_unknown_encoding_chardet(
     tmp_path: Path,
     capsys: pytest.CaptureFixture[str],
@@ -398,26 +399,69 @@ def test_ignore(
     tmp_path: Path,
     capsys: pytest.CaptureFixture[str],
 ) -> None:
-    """Test ignoring of files and directories."""
+    """Test ignoring of files and directories.
+
+    Test the following file hierarchy:
+
+        tmpdir
+        ├── good.txt
+        ├── bad.txt
+        └── ignoredir
+            └── subdir
+                └── bad.txt
+
+    """
     goodtxt = tmp_path / "good.txt"
     goodtxt.write_text("this file is okay")
-    assert cs.main(tmp_path) == 0
+    assert cs.main(d) == 0
     badtxt = tmp_path / "bad.txt"
     badtxt.write_text("abandonned")
     assert cs.main(tmp_path) == 1
     assert cs.main("--skip=bad*", tmp_path) == 0
+    assert cs.main("--skip=*bad*", tmp_path) == 0
     assert cs.main("--skip=bad.txt", tmp_path) == 0
-    subdir = tmp_path / "ignoredir"
+    assert cs.main("--skip=./bad.txt", tmp_path) == 0  # deprecated
+    subdir = tmp_path / "ignoredir" / "subdir"
     subdir.mkdir()
     (subdir / "bad.txt").write_text("abandonned")
     assert cs.main(tmp_path) == 2
-    assert cs.main("--skip=bad*", tmp_path) == 0
-    assert cs.main("--skip=*ignoredir*", tmp_path) == 1
-    assert cs.main("--skip=ignoredir", tmp_path) == 1
-    assert cs.main("--skip=*ignoredir/bad*", tmp_path) == 1
-    badjs = tmp_path / "bad.js"
-    copyfile(badtxt, badjs)
-    assert cs.main("--skip=*.js", goodtxt, badtxt, badjs) == 1
+    assert cs.main(goodtxt, badtxt) == 1
+    assert cs.main('--skip=*.txt', tmp_path) == 0
+    assert cs.main('--skip=*.txt', goodtxt, badtxt) == 0
+    assert cs.main('--skip=bad*', tmp_path) == 0
+    assert cs.main('--skip=*ignoredir*', tmp_path) == 1
+    assert cs.main('--skip=*gnoredir*', tmp_path) == 1
+    assert cs.main('--skip=ignoredir', tmp_path) == 1
+    assert cs.main('--skip=ignoredir/', tmp_path) == 1
+    assert cs.main('--skip=*ignoredir/subdir*', tmp_path) == 1
+    assert cs.main('--skip=*gnoredir/subdi*', tmp_path) == 1
+    assert cs.main('--skip=ignoredir/subdir', tmp_path) == 1
+    assert cs.main('--skip=ignoredir/subdir/', tmp_path) == 1
+    assert cs.main('--skip=*ignoredir/subdir/bad*', tmp_path) == 1
+    assert cs.main('--skip=ignoredir/subdir/bad.txt', tmp_path) == 1
+    assert cs.main('--skip=*subdir*', tmp_path) == 1
+    assert cs.main('--skip=*ubd*', tmp_path) == 1
+    assert cs.main('--skip=subdir', tmp_path) == 1
+    assert cs.main('--skip=*subdir/bad*', tmp_path) == 1
+    assert cs.main('--skip=subdir/bad*', tmp_path) == 2
+    # test deprecated syntax from outside "tmpdir"
+    assert cs.main('--skip=./ignoredir', tmp_path) == 2
+    assert cs.main('--skip=./ignoredir/subdir/', tmp_path) == 2
+    assert cs.main('--skip=./ignoredir/subdir/bad.txt', tmp_path) == 2
+    assert cs.main(f'--skip={d}/ignoredir', tmp_path) == 1
+    assert cs.main(f'--skip={d}/ignoredir/subdir/', tmp_path) == 1
+    assert cs.main(f'--skip={d}/ignoredir/subdir/bad.txt', tmp_path) == 1
+    # test deprecated syntax from inside "tmpdir"
+    cwd = os.getcwd()
+    os.chdir(tmpdir)
+    assert cs.main('--skip=./ignoredir') == 1
+    assert cs.main('--skip=./ignoredir/subdir/') == 1
+    assert cs.main('--skip=./ignoredir/subdir/bad.txt') == 1
+    assert cs.main('--skip=./ignoredir', '.') == 1
+    assert cs.main(f'--skip={d}/ignoredir') == 1
+    assert cs.main(f'--skip={d}/ignoredir/subdir/') == 1
+    assert cs.main(f'--skip={d}/ignoredir/subdir/bad.txt') == 1
+    os.chdir(cwd)
 
 
 def test_check_filename(