Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: improve file locking mechanism with atomic creation #455

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

fix: improve file locking mechanism with atomic creation

c9b63b0
Select commit
Loading
Failed to load commit list.
Open

fix: improve file locking mechanism with atomic creation #455

fix: improve file locking mechanism with atomic creation
c9b63b0
Select commit
Loading
Failed to load commit list.
qiniu-x / pylint failed Dec 5, 2024 in 36s

pylint found 2 issues related to your changes

This is the detailed log.

If you have any questions about this comment, feel free to raise an issue here.

Annotations

Check warning on line 151 in qiniu/http/regions_provider.py

See this annotation in the file changed.

@qiniu-x qiniu-x / pylint

qiniu/http/regions_provider.py#L151 

Consider explicitly re-raising using 'except FileExistsError as exc' and 'raise FileAlreadyLocked('File {0} already locked'.format(self._origin_file)) from exc' (raise-missing-from)

Check warning on line 151 in qiniu/http/regions_provider.py

See this annotation in the file changed.

@qiniu-x qiniu-x / pylint

qiniu/http/regions_provider.py#L151 

Formatting a regular string which could be an f-string (consider-using-f-string)
View more details on qiniu-x