Skip to content

Conversation

@tanbro
Copy link
Owner

@tanbro tanbro commented Dec 1, 2024

No description provided.

Liu Xue Yan added 6 commits November 30, 2024 23:29
- Remove safe_name function from utils.py
- This function was likely an early attempt at implementing a name sanitization function
- It has been unused since the project's initial commit
…ock mixins

- Rename TKey to KT in MySQL and PostgreSQL lock mixin classes
- Update type hints in __init__ methods to use the new type variable
- Add type hints for 'key' parameter in lock classes
- Simplify key usage by removing unnecessary underscores
- Update MysqlAsyncSadLock, PostgresqlAsyncSadLock, MysqlSadLock, and PostgresqlSadLock classes
- Restore async lock classes and factory function
- Update imports and exports in various modules
- Adjust test files to use the new async lock functionality
- Remove unnecessary asyncio-specific files and code
- Add LRU cache to find_lock_class function in utils.py
- Remove unnecessary type annotations and imports
- Simplify lock class retrieval logic
- Add `clean` option to documentation build process
- Remove old apidocs before generating new ones
- Update apidocs to reflect recent code changes
- Remove unused async package from apidocs
@codecov
Copy link

codecov bot commented Dec 1, 2024

Codecov Report

Attention: Patch coverage is 97.43590% with 4 lines in your changes missing coverage. Please review.

Project coverage is 95.27%. Comparing base (03d00f9) to head (cea3da3).
Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
src/sqlalchemy_dlock/lock/postgresql.py 96.07% 2 Missing ⚠️
src/sqlalchemy_dlock/factory.py 94.11% 1 Missing ⚠️
src/sqlalchemy_dlock/lock/base.py 96.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   95.12%   95.27%   +0.15%     
==========================================
  Files          18       12       -6     
  Lines         410      381      -29     
==========================================
- Hits          390      363      -27     
+ Misses         20       18       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tanbro tanbro merged commit 2992161 into main Dec 1, 2024
17 checks passed
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