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.

Sign up for GitHub

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

Jump to bottom

Add vulnerability severity data to command output. #90

Merged
merged 9 commits into from
Jul 26, 2023
Merged

Add vulnerability severity data to command output. #90

merged 9 commits into from
Jul 26, 2023

Conversation

iamdharmesh
Copy link
Member

Description of the Change

PR adds severity data of vulnerability in command output.

image

Closes #85

How to test the Change

  1. Configure this scanner repo and configure API provider
  2. Run commands like wp vuln status, wp vuln theme-status etc... and make sure severity details are there.

Changelog Entry

Added - Vulnerability severity data to command output.

Credits

Props @iamdharmesh

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@iamdharmesh iamdharmesh self-assigned this Mar 31, 2023
@iamdharmesh iamdharmesh added this to the 1.2.0 milestone Mar 31, 2023
@iamdharmesh iamdharmesh marked this pull request as draft March 31, 2023 15:01
@jeffpaul
Copy link
Member

jeffpaul commented Apr 3, 2023

@iamdharmesh is this ready to send out for review or still work in progress?

@iamdharmesh
Copy link
Member Author

@iamdharmesh is this ready to send out for review or still work in progress?

@jeffpaul work is done, but Tests for Wordfence are failing. So, I have made a PR draft until I figure out the root cause for the failure and sort it out.

@iamdharmesh iamdharmesh marked this pull request as ready for review July 4, 2023 05:18
@iamdharmesh iamdharmesh requested review from a team and peterwilsoncc and removed request for a team July 4, 2023 05:19
This was referenced Jul 6, 2023
Closed
Merged
@jeffpaul jeffpaul marked this pull request as draft July 14, 2023 18:26
@jeffpaul jeffpaul removed the request for review from peterwilsoncc July 14, 2023 18:26
@iamdharmesh iamdharmesh marked this pull request as ready for review July 17, 2023 05:11
@iamdharmesh iamdharmesh marked this pull request as draft July 17, 2023 05:12
@iamdharmesh iamdharmesh marked this pull request as ready for review July 17, 2023 05:13
@iamdharmesh
Copy link
Member Author

@jeffpaul this is ready for review now.

@iamdharmesh iamdharmesh requested review from a team and Sidsector9 and removed request for a team July 17, 2023 05:13
Sidsector9
Sidsector9 approved these changes Jul 25, 2023
View reviewed changes
Copy link
Member

@Sidsector9 Sidsector9 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@iamdharmesh LGTM 👍

@iamdharmesh iamdharmesh merged commit fe0ccca into develop Jul 26, 2023
3 of 4 checks passed
@iamdharmesh iamdharmesh deleted the enhancement/85 branch July 26, 2023 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sidsector9 Sidsector9 Sidsector9 approved these changes

Assignees

@iamdharmesh iamdharmesh

Labels
None yet
Projects
None yet
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.

Add Severity information
3 participants
@iamdharmesh @jeffpaul @Sidsector9