From 3abf6929347281daa372a41193f8dfd7380b573d Mon Sep 17 00:00:00 2001 From: skyflow-himanshu Date: Mon, 8 Dec 2025 13:54:05 +0530 Subject: [PATCH 1/2] SK-2421: integrate codespell into ci cd --- .codespellrc | 13 +++++++++++++ .github/workflows/shared-tests.yml | 6 ++++++ CHANGELOG.md | 2 +- .../signed_token_generation_example.py | 2 +- setup.py | 5 +++++ 5 files changed, 26 insertions(+), 2 deletions(-) create mode 100644 .codespellrc diff --git a/.codespellrc b/.codespellrc new file mode 100644 index 00000000..261197eb --- /dev/null +++ b/.codespellrc @@ -0,0 +1,13 @@ +[codespell] +# custom words file +ignore-words-list = Skyflow,skyflow,skyflowapi,skyflowapis,deidentify,reidentify,detokenize,upsert,upserting,binlookup,byot,creds,fpe,devsecops,formdata,vaultid,dotenv,usecwd,runid,dateutil,Homogenous + +# Skip these files and folders +skip = .git,.venv,venv,env,__pycache__,*.pyc,*.egg-info,dist,build,.idea,.vscode,*.log,requirements.txt,./skyflow/generated,setup.py + +# If you want to verify it is working, you can uncomment this line to see what files it checks +# count = + +# Ignore lines that match these patterns +# This ignores lines with long alphanumeric strings or UUIDs +ignore-regex = \b[A-Z0-9]{20,}\b|[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}|\b[A-Za-z0-9_]{7,}\b(?=]) \ No newline at end of file diff --git a/.github/workflows/shared-tests.yml b/.github/workflows/shared-tests.yml index 73253e3c..dfe6b37a 100644 --- a/.github/workflows/shared-tests.yml +++ b/.github/workflows/shared-tests.yml @@ -29,6 +29,12 @@ jobs: pip install --upgrade pip setuptools wheel python setup.py sdist bdist_wheel pip install dist/skyflow-*.whl + + - name: Install Dev Dependencies + run: pip install ".[dev]" + + - name: Run Spell Check + run: codespell - name: 'Run Tests' run: | diff --git a/CHANGELOG.md b/CHANGELOG.md index e2b5abed..6f564510 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -89,7 +89,7 @@ All notable changes to this project will be documented in this file. ### Fixed -- Request headers not getting overriden due to case sensitivity +- Request headers not getting overridden due to case sensitivity ## [1.5.0] - 2022-03-22 diff --git a/samples/service_account/signed_token_generation_example.py b/samples/service_account/signed_token_generation_example.py index 1c97a1fb..32140ada 100644 --- a/samples/service_account/signed_token_generation_example.py +++ b/samples/service_account/signed_token_generation_example.py @@ -19,7 +19,7 @@ options = { - 'ctx': 'CONTEX_ID', + 'ctx': 'CONTEXT_ID', 'data_tokens': ['DATA_TOKEN1', 'DATA_TOKEN2'], 'time_to_live': 90, # in seconds } diff --git a/setup.py b/setup.py index 9f06860f..b27bb0bc 100644 --- a/setup.py +++ b/setup.py @@ -33,5 +33,10 @@ 'python-dotenv~=1.0.1', 'httpx' ], + extras_require={ + 'dev': [ + 'codespell', + ] + }, python_requires=">=3.8", ) From 10e39d0f02facb313b6f81f67a2ce56ba5411fb4 Mon Sep 17 00:00:00 2001 From: skyflow-himanshu Date: Tue, 9 Dec 2025 17:51:06 +0530 Subject: [PATCH 2/2] SK-2421: refactor shared-tests yml file to integrate codespell --- .github/workflows/shared-tests.yml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/.github/workflows/shared-tests.yml b/.github/workflows/shared-tests.yml index dfe6b37a..9ea5726b 100644 --- a/.github/workflows/shared-tests.yml +++ b/.github/workflows/shared-tests.yml @@ -29,9 +29,7 @@ jobs: pip install --upgrade pip setuptools wheel python setup.py sdist bdist_wheel pip install dist/skyflow-*.whl - - - name: Install Dev Dependencies - run: pip install ".[dev]" + pip install ".[dev]" - name: Run Spell Check run: codespell