Rename skale_te to t-encrypt#293
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request renames the threshold encryption Python package from skale_te/skale-te to t_encrypt/t-encrypt, making the naming more concise and consistent with the JavaScript package naming convention already in use. The changes include updating the C++ shared library name, Python package structure, environment variables, exception classes, and all related references in build configurations and CI/CD workflows.
Changes:
- Renamed C++ shared library from
libskale_te_python.sotolibt_encrypt_python.soin CMake configuration - Restructured Python package from
python/skale_te/topython/t_encrypt/with updated exception class names - Updated environment variable from
SKALE_TE_LIB_PATHtoT_ENCRYPT_LIB_PATHacross setup scripts and CI/CD workflows - Removed trailing whitespace in multiple files as style improvements
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| threshold_encryption/CMakeLists.txt | Renamed Python shared library target from skale_te_python to t_encrypt_python and removed trailing whitespace |
| python/t_encrypt/exceptions.py | Added new exception classes with TEncryptError as base (replacing SkaleTEError) |
| python/t_encrypt/core.py | Updated library path to reference libt_encrypt_python.so instead of libskale_te_python.so |
| python/t_encrypt/init.py | Updated module docstring and exception imports to use new TEncryptError naming |
| python/skale_te/exceptions.py | Removed old exception file as part of package restructuring |
| python/setup_t_encrypt.py | Updated package name to t-encrypt, module name to t_encrypt, and environment variable to T_ENCRYPT_LIB_PATH |
| python/setup.sh | Updated script to reference setup_t_encrypt.py and new library/package names |
| python/MANIFEST.in | Updated to include t_encrypt directory instead of skale_te |
| .github/workflows/publish.yml | Updated all references to use new package name, library name, and environment variable; removed trailing whitespace |
olehnikolaiev
approved these changes
Feb 13, 2026
badrogger
approved these changes
Feb 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rename skale_te to t-encrypt
Restucture the path to t-encrypt Python