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.

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

objects.artifact delete function docstring return type is incorrect #207

Open
kkaja123 opened this issue Jan 27, 2025 · 0 comments
Open

Comments

@kkaja123
Copy link

kkaja123 commented Jan 27, 2025

The docstring says that the ArtifactoryArtifact delete() method returns a bool, but it doesn't actually return a bool. self._delete returns an HTTP Response, so maybe that should be used to determine whether or not to return True or False. Otherwise, if delete shouldn't return anything, then remove the line from the docstring.

def delete(self, artifact_path: str) -> None:
"""
:param artifact_path: Path to file in Artifactory
:return: bool
"""

@kkaja123 kkaja123 changed the title object.artifact delete function docstring return type is incorrect objects.artifact delete function docstring return type is incorrect Jan 27, 2025
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

No branches or pull requests

1 participant