Merge pull request #41 from Jibola/run-lint

Make lint fixes provided by pre-commit

Author: Jibola

.evergreen/config.yml

@@ -187,7 +187,7 @@ buildvariants:
       - rhel87-small
     tasks:
       - name: test-llama-index
-  
+
   - name: test-docarray-rhel
     display_name: DocArray RHEL
     expansions:
@@ -210,4 +210,4 @@ buildvariants:
     run_on:
       - rhel87-small
     tasks:
-      - name: test-autogen
+      - name: test-autogen

.evergreen/provision-atlas.sh

@@ -7,9 +7,7 @@ PYTHON_BINARY=$(find_python3)
 # Should be called from src
 EVERGREEN_PATH=$(pwd)/.evergreen
 TARGET_DIR=$(pwd)/$DIR
-PING_ATLAS=$EVERGREEN_PATH/ping_atlas.py
 SCAFFOLD_SCRIPT=$EVERGREEN_PATH/scaffold_atlas.py
-DEPLOYMENT_NAME=$DIR
 
 set -ex
 mkdir atlas

.evergreen/scaffold_atlas.py

@@ -36,7 +36,7 @@ def upload_data(db: Database, filename: Path) -> None:
         loaded_collection = json.load(f)
 
     logger.info(
-        "Loading %s to Atlas database %s in colleciton %s",
+        "Loading %s to Atlas database %s in collection %s",
         filename.name,
         db.name,
         collection_name,

.evergreen/utils.sh

@@ -80,7 +80,7 @@ setup_local_atlas() {
             *)
             echo "Unrecognized state $STATE"
             sleep 1
-        esac      
+        esac
     done
 
     echo "container did not get healthy within 120 seconds, quitting"
@@ -92,14 +92,14 @@ setup_local_atlas() {
     wait "$CONTAINER_ID"
     EXPOSED_PORT=$(podman inspect --format='{{ (index (index .NetworkSettings.Ports "27017/tcp") 0).HostPort }}' "$CONTAINER_ID")
     export CONN_STRING="mongodb://127.0.0.1:$EXPOSED_PORT/?directConnection=true"
+    # shellcheck disable=SC2154
     echo "CONN_STRING=mongodb://127.0.0.1:$EXPOSED_PORT/?directConnection=true" > $workdir/src/.evergreen/.local_atlas_uri
 }
 
 fetch_local_atlas_uri() {
+    # shellcheck disable=SC2154
     . $workdir/src/.evergreen/.local_atlas_uri
 
     export CONN_STRING=$CONN_STRING
     echo "$CONN_STRING"
 }
-
-

.github/workflows/lint.yml

@@ -26,4 +26,4 @@ jobs:
           python -m pip install -U pip pre-commit
       - name: Run linters
         run: |
-          pre-commit run --hook-stage=manual --all-files
+          pre-commit run --hook-stage=manual --all-files

.gitignore

@@ -49,4 +49,3 @@ xunit-results/
 
 # Miscellaneous
 .DS_Store
-

README.md

@@ -30,7 +30,7 @@ Within each subdirectory you should expect to have:
 The general layout of this repo looks like this:
 
 ```bash
-├── LICENSE                                     # License Agreeement
+├── LICENSE                                     # License Agreement
 ├── README.md                                   # This Document
 ├── langchain-python                            # Folder scoped for one Integration
 │   └── run.sh                                  # Script that executes test

autogen/run.sh

@@ -6,18 +6,20 @@
 
 set -x
 
+# shellcheck disable=SC2154
 . $workdir/src/.evergreen/utils.sh
 PYTHON_BINARY=$(find_python3)
 $PYTHON_BINARY -c "import sys; print(f'Python version found: {sys.version_info}')"
 
 # Create and activate an isolated python venv environment
 $PYTHON_BINARY -m venv venv
-source venv/bin/activate
+. venv/bin/activate
 #  Install autogen with extras
 $PYTHON_BINARY -m pip install .[test,"retrievechat-mongodb"]
 
 
 # Run tests. Sensitive variables in Evergreen come from Evergreen project: ai-ml-pipeline-testing/
+# shellcheck disable=SC2154
 MONGODB_URI=$autogen_mongodb_uri \
 MONGODB_DATABASE="autogen_test_db" \
 $PYTHON_BINARY -m pytest -v test/agentchat/contrib/vectordb/test_mongodb.py

chatgpt-retrieval-plugin/run.sh

@@ -4,6 +4,7 @@
 
 set -x
 
+# shellcheck disable=SC2154
 . $workdir/src/.evergreen/utils.sh
 
 PYTHON_BINARY=$(find_python3)
@@ -14,13 +15,14 @@ $PYTHON_BINARY -m pip install -U pip poetry
 # Create a package specific poetry environment
 $PYTHON_BINARY -m poetry env use $PYTHON_BINARY
 # Activate the poetry env, which itself does not include poetry
-source $($PYTHON_BINARY -m poetry env info --path)/bin/activate
+. "$($PYTHON_BINARY -m poetry env info --path)/bin/activate"
 # Recreate the poetry lock file
 $PYTHON_BINARY -m poetry lock --no-update
 # Install from pyproject.toml into package specific environment
 $PYTHON_BINARY -m poetry install --with dev
 
 # Run tests. Sensitive variables in Evergreen come from Evergeen project: ai-ml-pipeline-testing/
+# shellcheck disable=SC2154
 OPENAI_API_KEY=$openai_api_key \
 DATASTORE="mongodb-atlas" \
 BEARER_TOKEN="staylowandkeepmoving" \

docarray/run.sh

@@ -5,13 +5,14 @@
 
 set -x
 
+# shellcheck disable=SC2154
 . $workdir/src/.evergreen/utils.sh
 PYTHON_BINARY=$(find_python3)
 $PYTHON_BINARY -c "import sys; print(f'Python version found: {sys.version_info}')"
 
 # Create and activate an isolated python venv environment
 $PYTHON_BINARY -m venv venv
-source venv/bin/activate
+. venv/bin/activate
 # Install Poetry
 pip install -U pip poetry
 # Recreate the poetry lock file
@@ -21,6 +22,7 @@ poetry install --with dev --extras mongo
 
 
 # Run tests. Sensitive variables in Evergreen come from Evergeen project: ai-ml-pipeline-testing/
+# shellcheck disable=SC2154
 MONGODB_URI=$docarray_mongodb_uri \
 MONGODB_DATABASE="docarray_test_db" \
-pytest -v tests/index/mongo_atlas
+pytest -v tests/index/mongo_atlas