Migrating to a devcontainer

Also bumped version.

.devcontainer/devcontainer.json

@@ -0,0 +1,22 @@
+// For format details, see https://aka.ms/devcontainer.json. For config options, see the
+// README at: https://github.com/devcontainers/templates/tree/main/src/python
+{
+	"name": "Python 3",
+	// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
+	"image": "mcr.microsoft.com/devcontainers/python:1-3.12-bullseye"
+
+	// Features to add to the dev container. More info: https://containers.dev/features.
+	// "features": {},
+
+	// Use 'forwardPorts' to make a list of ports inside the container available locally.
+	// "forwardPorts": [],
+
+	// Use 'postCreateCommand' to run commands after the container is created.
+	// "postCreateCommand": "pip3 install --user -r requirements.txt",
+
+	// Configure tool-specific properties.
+	// "customizations": {},
+
+	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
+	// "remoteUser": "root"
+}

Jenkinsfile

@@ -1,19 +1,20 @@
 pipeline {
-    agent any
+    agent {
-    stages {
+        kubernetes {
-        stage("clean") {
+            cloud 'rsys-devel'
-            steps {
+            defaultContainer 'pip'
-                sh 'rm -rf ./dist/*'
+            inheritFrom 'pip'
-            }
         }
+    }
+    stages {
         stage("install") {
             steps {
-                sh 'mamba env update --file environment.yml'
+                sh 'pip install -r requirements.txt'
-                sh 'echo "mamba activate renamebycsv" >> ~/.bashrc'
             }
         }
         stage("build") {
             steps {
+                sh 'rm -rf ./dist/*'
                 sh "python -m build"
             }
         }
@@ -25,17 +26,18 @@ pipeline {
         }
         stage("archive") {
             steps {
-                archiveArtifacts artifacts: 'dist/*.tar.gz, dist/*.whl'
+                archiveArtifacts artifacts: 'dist/*.tar.gz, dist/*.whl', fingerprint: true, followSymlinks: false, onlyIfSuccessful: true
             }
         }
         stage("publish package") {
+            environment {
+                CREDS = credentials('rs-git-package-registry-ydeng')
+            }
             when {
                 branch '**/master'
             }
             steps {
-                withCredentials([usernamePassword(credentialsId: 'rs-git-package-registry-ydeng', passwordVariable: 'PASS', usernameVariable: 'USER')]) {
+                sh returnStatus: true, script: 'python -m twine upload --repository-url https://git.reslate.systems/api/packages/${CREDS_USR}/pypi -u ${CREDS_USR} -p ${CREDS_PSW} --non-interactive --disable-progress-bar --verbose dist/*'
-                    sh returnStatus: true, script: 'python -m twine upload --repository-url https://git.reslate.systems/api/packages/${USER}/pypi -u ${USER} -p ${PASS} --non-interactive --disable-progress-bar --verbose dist/*'
-                }
             }
         }
     }

README.md

@@ -24,11 +24,19 @@ This program makes heavy use of REGEX, also known as Regular Expression to give
  - Where it differs is the ability to use one REGEX string to match many strings.
    - i.e, the REGEX "`abc\d+`" will match with "`abc1`", "`abc2`", "`abc12`", but not "`ac12`" or "`abc`".
  - Many characters can be used as normal and will match a string literally (character for character), but some will be treated as special characters (such as the previously used `\`, which indicates that the letter afterwards should be treated specially, such as a token)
-   - Common tokens to be aware of: `.` for any character, `\d` for single digits, `\w` for word characters, `\s` for space characters (tabs, spaces, linebreaks, etc.). Tokens can be repeated by using `+`, indicating "one or more", `*` indicating "none or more".
+   - Common tokens to be aware of: `.` for any character, `\d` for single digits, `\w` for word characters, `\s` for space characters (tabs, spaces, linebreaks, etc.). Tokens can be repeated by using `+`, indicating "one or more", `*` indicating "none or more". If you want to match something that is read as a token by default, such as `.`, or `+`, using the `\` in front of it will cause it to match `.` literally, i.e, `1\.2` matches `1.3`, but not `123`, `1a3`, etc.
  - A capture group is a way of "selecting" a part of a text and is formed by using `(` and `)` around the REGEX that should be selected.
 
  Now for a few examples:
 
-Let's say we have files `run325-a-1.vcf`, `run326-b-2.vcf`, and `run327-b-3.vcf`. If we know that all that matters is the `1` after the `run[numbers]-[character]-`, we can write `run\d+-\w-(\d).vcf` which will match with all 3 of the above examples, and select the last digit. The program can then use a given CSV to look up the selected digits and replace the name with what is given by the CSV.
+Let's say we have files `run325-a-1.vcf`, `run326-b-2.vcf`, and `run327-b-3.vcf`. If we know that all that matters is the `1` after the `run[numbers]-[character]-`, we can write `run\d+-\w-(\d)\.vcf` which will match with all 3 of the above examples, and select the last digit. The program can then use a given CSV to look up the selected digits and replace the name with what is given by the CSV.
 
 For learning and testing your own REGEX, checkout [regex101.com](https://regex101.com/), which allows you to write the strings that you're trying to match, and the REGEX. It will show you live which parts of the strings match to what, if any parts match.
+
+## Not Working?
+
+If the program is not working the way you would like it, try running the program in `-v DEBUG` mode which increases verbosity. Typically, files not being renamed can be attributed to one of two problems:
+
+1. It's looking in the wrong directory. The solution would be to double check that the directory it's looking in (printed by the program each run) is correct. If not, try adding quotes around the path in the command line.
+
+2. The provided REGEX pattern isn't matching to any of the files. In this case, test one or two of the files at [regex101.com](https://regex101.com/) with your pattern.

environment.yml

@@ -9,3 +9,4 @@ dependencies:
   - setuptools=67.6
   - twine=4.0
   - cryptography=38.0.4
+prefix: ./env

renamebycsv/cli.py

@@ -7,16 +7,17 @@ from renamebycsv.renamer import find_all_candidates, rename_by_csv
 
 
 def run(args):
-    candidates = find_all_candidates(args.input_dir, args.regex, args.recursive)
+    candidates = find_all_candidates(args.input_dir, args.pattern, args.recursive)
-    rename_by_csv(
+    if len(candidates):
-        args.csv,
+        rename_by_csv(
-        candidates,
+            args.csv,
-        args.current,
+            candidates,
-        args.become,
+            args.current,
-        args.dry,
+            args.become,
-        args.extension,
+            args.dry,
-        args.keep_extension,
+            args.extension,
-    )
+            args.keep_extension,
+        )
 
 
 def main():

renamebycsv/renamer.py

@@ -6,6 +6,12 @@ from typing import Iterable
 
 
 def find_all_candidates(input_dir: str, regex: str, recursive: bool):
+    logging.info(
+        'Searching "%s" for files that match "%s" %s',
+        input_dir,
+        regex,
+        "recursively" if recursive else "non-recursively",
+    )
     results = []
     for subitem in os.listdir(input_dir):
         subitem_path = os.path.join(input_dir, subitem)
@@ -19,6 +25,15 @@ def find_all_candidates(input_dir: str, regex: str, recursive: bool):
                 continue
             results.append((subitem_path, subitem, match))
             logging.debug(f'Collecting "{subitem}"...')
+    if len(results) < 1:
+        logging.info(
+            'No results found matching "%s" in "%s". Please double check your REGEX '
+            "pattern and directory being searched.",
+            regex,
+            input_dir,
+        )
+    else:
+        logging.info("Collected %d files to rename.", len(results))
     return results
 
 
@@ -38,6 +53,10 @@ def rename_by_csv(
         become_col_ind = None
         for row in reader:
             if current_col_ind is None and become_col_ind is None:
+                if current not in row:
+                    logging.error("\"%s\" not in header %s.", current, list(row))
+                if become not in row:
+                    logging.error("\"%s\" not in header %s.", become, list(row))
                 current_col_ind = row.index(current)
                 become_col_ind = row.index(become)
                 continue
@@ -45,6 +64,7 @@ def rename_by_csv(
                 row[current_col_ind] in replacement_dict
                 and replacement_dict[row[current_col_ind]] != row[become_col_ind]
             ):
+                # Check if there's a duplicate key for different values.
                 raise Exception("Duplicate current key.")
             replacement_dict[row[current_col_ind]] = row[become_col_ind]
     for subitem_path, subitem, match in candidates:
@@ -67,7 +87,7 @@ def rename_by_csv(
         logging.info(f'Will rename "{original}" to "{os.path.basename(objective)}"')
         if os.path.exists(objective):
             logging.error(
-                f'Path at "{objective}" exists, not continuing. '
+                f'Path at "{objective}" already exists, not continuing. '
                 "Use -f to overwrite instead of stopping."
             )
             exit(1)

requirements.txt

@@ -0,0 +1,5 @@
+build
+pytest
+setuptools
+twine
+cryptography

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = renamebycsv
-version = 0.0.6
+version = 0.0.8
 
 [options]
 packages = renamebycsv