Skip to content

[WIP] feat: update for new file system implementation #75

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.

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

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

Conversation

harry97uk
Copy link
Contributor

No description provided.

@harry97uk harry97uk requested a review from CodeBooster97 April 23, 2025 16:31
Copy link

codeclimate bot commented Apr 23, 2025

❌ 5 blocking issues (5 total)

Tool Category Rule Count
radarlint-python Lint Replace this generic exception class with a more specific one. 3
radarlint-python Lint Rename this parameter "fileName" to match the regular expression ^[_a-z][a-z0-9_]*$. 1
radarlint-python Lint Define a constant instead of duplicating this literal "https://v2\.api\.audio/v3" 9 times. 1

This is from Qlty Cloud, the successor to Code Climate Quality. Learn more.

self.created_at = response["createdAt"]
self.status = response["status"]

def download(self, fileName: str, path: str = "./") -> None:
Copy link

Choose a reason for hiding this comment

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

Rename this parameter "fileName" to match the regular expression ^[_a-z][a-z0-9_]*$. [radarlint-python:python.S117]

fileName = self.filePath.split("/")[-1]
raise Exception("Please supply a valid file name")
if not self.url:
raise Exception("No URL found for this file")
Copy link

Choose a reason for hiding this comment

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

Replace this generic exception class with a more specific one. [radarlint-python:python.S112]

) -> Item:
if not os.path.isfile(localPath):
if not local_path:
raise Exception("Please supply a localPath (path to your local file)")
Copy link

Choose a reason for hiding this comment

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

Replace this generic exception class with a more specific one. [radarlint-python:python.S112]

"metadata": metadata,
}
if not file_name:
raise Exception("Please supply a valid file name")
Copy link

Choose a reason for hiding this comment

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

Replace this generic exception class with a more specific one. [radarlint-python:python.S112]

rtype=RequestTypes.POST,
route="file/create-upload-url",
json=payload,
overwrite_base_url="https://v2.api.audio/v3",
Copy link

Choose a reason for hiding this comment

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

Define a constant instead of duplicating this literal "https://v2\.api\.audio/v3" 9 times. [radarlint-python:python.S1192]

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.

1 participant