From c9cb690d7013296f344d8af16560ddf4bdf0e5bb Mon Sep 17 00:00:00 2001 From: texastony <5892063+texastony@users.noreply.github.com> Date: Wed, 14 Jan 2026 14:42:34 -0600 Subject: [PATCH 1/5] chore[dev]: replace poetry with UV in README --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index c24d1796..b3b08fb8 100644 --- a/README.md +++ b/README.md @@ -7,7 +7,7 @@ This library provides an S3 client that supports client-side encryption. ### Prerequisites - Python 3.11 or higher -- [Poetry](https://python-poetry.org/) for dependency management +- [uv](https://github.com/astral-sh/uv) for package and project management ### Setup From df4c0c81da8fee960fb4a7c5826601d66ee755a2 Mon Sep 17 00:00:00 2001 From: texastony <5892063+texastony@users.noreply.github.com> Date: Wed, 14 Jan 2026 14:47:05 -0600 Subject: [PATCH 2/5] fix[ci]: run-tests in main.yml needs permissions --- .github/workflows/main.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index e10b7d0d..675a0099 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -2,7 +2,7 @@ name: Main Workflow on: push: - branches: [ main ] + branches: [ staging ] pull_request: workflow_dispatch: inputs: @@ -22,4 +22,7 @@ jobs: uses: ./.github/workflows/test.yml with: python-version: ${{ inputs.python-version || '3.11' }} + permissions: + id-token: write + contents: read secrets: inherit From 527346e09d25d03524bef0cdbbd3294e47c63684 Mon Sep 17 00:00:00 2001 From: Tony Knapp <5892063+texastony@users.noreply.github.com> Date: Wed, 14 Jan 2026 15:28:47 -0600 Subject: [PATCH 3/5] chore[dev]: Add PR helper to instruction (#122) --- README.md | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/README.md b/README.md index b3b08fb8..68426094 100644 --- a/README.md +++ b/README.md @@ -73,3 +73,14 @@ Common Flake8 issues in the codebase include: - **Code complexity** (C901): Refactor complex functions When contributing to this project, please try to fix linting issues in the files you modify. + +### Pull Request Command +While this project is in development, +it is useful to use `gh pr` to create the pull-requests, +so they can be associated with the GitHub project, +as compared to the FireEgg event. + +```sh +gh pr create -B staging -p "S3EC-Python" -f +``` + From f4528c89cf9be5e5bba31e3b3c38a14f26a42aed Mon Sep 17 00:00:00 2001 From: texastony <5892063+texastony@users.noreply.github.com> Date: Sat, 24 Jan 2026 13:57:57 -0600 Subject: [PATCH 4/5] chore: resolve PR feedback --- README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/README.md b/README.md index 68426094..edcfd4ff 100644 --- a/README.md +++ b/README.md @@ -77,8 +77,7 @@ When contributing to this project, please try to fix linting issues in the files ### Pull Request Command While this project is in development, it is useful to use `gh pr` to create the pull-requests, -so they can be associated with the GitHub project, -as compared to the FireEgg event. +so they can be associated with the GitHub project. ```sh gh pr create -B staging -p "S3EC-Python" -f From ab1bbd3c97d323363391573d7a5db51d5ca9ff81 Mon Sep 17 00:00:00 2001 From: texastony <5892063+texastony@users.noreply.github.com> Date: Sat, 24 Jan 2026 14:11:01 -0600 Subject: [PATCH 5/5] chore: suppress docstring and line length linting rules for test files Add per-file-ignores in pyproject.toml to suppress D100-D104 (missing docstrings) and E501 (line length) violations in test files, reducing lint errors from 30 to 3. --- pyproject.toml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index 883b6914..187b99a2 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -19,7 +19,8 @@ test = [ "pytest>=8.4.1", ] dev = [ - "black>=24.3.0", + # black >= 26 requires a reformat + "black>=24.3.0,<26.0.0", "ruff>=0.3.0", ] @@ -52,3 +53,6 @@ max-complexity = 10 [tool.ruff.lint.isort] known-first-party = ["s3_encryption"] + +[tool.ruff.lint.per-file-ignores] +"test/**/*.py" = ["D100", "D101", "D102", "D103", "D104", "E501"]