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

Update README.md for aql example #436

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

jandvorak-sol60279
Copy link
Contributor

@jandvorak-sol60279 jandvorak-sol60279 commented Nov 23, 2023

added "type" key for aql example for aql results to work correctly with from_aql method for ArtifactoryPath

if "type" isnt included in aql response dict(s) it will raise ArtifactoryException and thus example isnt working

    def from_aql(self, result):
        """
        Convert raw AQL result to pathlib object
        :param result: ONE raw result
        :return:
        """
        result_type = result.get("type")
        if result_type not in ("file", "folder"):
            raise ArtifactoryException(
                f"Path object with type '{result_type}' doesn't support. File or folder only"
            )

added "type" key to  include for aql results to work correctly with from_aql method for ArtifactoryPath
@allburov allburov merged commit 9a1432f into devopshq:master Nov 24, 2023
5 checks passed
@allburov
Copy link
Member

@jandvorak-sol60279 thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants