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

Added aditional Parameter for multiple Version Info. #85

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

35C4n0r
Copy link

@35C4n0r 35C4n0r commented Dec 23, 2022

Closes: #64
Added an optional parameter all_versions which is False by Default, If passed True returns PackageInfo for all Versions.

Signed-off-by: Jay [email protected]

… If passed True returns PackageInfo for all Versions.

Signed-off-by: Jay <[email protected]>
Signed-off-by: Jay <[email protected]>
@35C4n0r
Copy link
Author

35C4n0r commented Dec 26, 2022

@pombredanne please review this once.

**version_purl.to_dict(),
)

if all_versions:
Copy link
Member

Choose a reason for hiding this comment

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

Please check the invert of it, this will help in removing indentation by one level.

Copy link
Author

Choose a reason for hiding this comment

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

@TG1999 can you please tell what is the meaning of invert here and what to do.

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.

Only return requested versions...
2 participants