Skip to content

feat(eng-2374): Add policy package query string#118

Merged
Leeoc merged 4 commits into
masterfrom
eng-2374/policy-package-search-string-cli
May 18, 2023
Merged

feat(eng-2374): Add policy package query string#118
Leeoc merged 4 commits into
masterfrom
eng-2374/policy-package-search-string-cli

Conversation

@Leeoc
Copy link
Copy Markdown
Contributor

@Leeoc Leeoc commented May 17, 2023

Adds package_query_string to the existing license and vulnerability policy CLI's

image

Comment thread cloudsmith_cli/data/config.ini Outdated
@Leeoc Leeoc force-pushed the eng-2374/policy-package-search-string-cli branch from 4ffe4d1 to 629abe7 Compare May 17, 2023 11:33
@Leeoc Leeoc force-pushed the eng-2374/policy-package-search-string-cli branch from 629abe7 to b78ed8f Compare May 17, 2023 11:38
@Leeoc Leeoc marked this pull request as ready for review May 17, 2023 11:46
Comment thread requirements/development.py3.txt Outdated
@chrisimcevoy
Copy link
Copy Markdown
Collaborator

I wonder do we need to start worrying about the length of some of these values when "pretty" printing.

Thinking if someone had a particularly cumbersome package_query_string, or a boatload of spdx_identifiers in a license policy?

For non-pretty output, we should show it all though.

Comment thread cloudsmith_cli/cli/tests/commands/policy/test_vulnerability.py
Comment thread cloudsmith_cli/cli/tests/commands/policy/test_licence.py
@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit 12ab94f and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 90.9% (50% is the threshold).

This pull request will bring the total coverage in the repository to 56.9% (0.1% change).

View more on Code Climate.

Copy link
Copy Markdown
Collaborator

@chrisimcevoy chrisimcevoy left a comment

Choose a reason for hiding this comment

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

Nice one! LGTM.

@Leeoc Leeoc merged commit fb622e2 into master May 18, 2023
@Leeoc Leeoc deleted the eng-2374/policy-package-search-string-cli branch May 18, 2023 08:55
@Leeoc Leeoc mentioned this pull request May 18, 2023
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