Skip to content

feat: Python EncryptedTable impl and tests#1895

Merged
lucasmcdonald3 merged 11 commits into
python-reviewedfrom
python-table
Jun 3, 2025
Merged

feat: Python EncryptedTable impl and tests#1895
lucasmcdonald3 merged 11 commits into
python-reviewedfrom
python-table

Conversation

@lucasmcdonald3

@lucasmcdonald3 lucasmcdonald3 commented May 14, 2025

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

Snippet of Python DBESDK to include only the EncryptedTable and tests.

CI doesn't run here; see #1794 for CI and for other files.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@lucasmcdonald3 lucasmcdonald3 changed the title Python: EncryptedTable impl and tests feat: Python EncryptedTable impl and tests May 14, 2025
@lucasmcdonald3 lucasmcdonald3 changed the base branch from main to python-reviewed May 14, 2025 23:30
Lucas McDonald added 3 commits May 19, 2025 15:23
@lucasmcdonald3 lucasmcdonald3 marked this pull request as ready for review May 21, 2025 16:34
@lucasmcdonald3 lucasmcdonald3 requested a review from a team as a code owner May 21, 2025 16:34
@lucasmcdonald3

Copy link
Copy Markdown
Contributor Author

Moving back to draft while I debug test vectors

@lucasmcdonald3 lucasmcdonald3 marked this pull request as draft May 28, 2025 21:06
@lucasmcdonald3 lucasmcdonald3 marked this pull request as ready for review May 29, 2025 22:09

@ajewellamz ajewellamz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lucasmcdonald3 lucasmcdonald3 merged commit 33c173f into python-reviewed Jun 3, 2025
3 checks passed
@lucasmcdonald3 lucasmcdonald3 deleted the python-table branch June 3, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants