Generate ulimit output - Githubissues

ChrisR2487 commented 1 year ago

Preflight includes information on user resource limits.

Describe how the desired outcome above has been achieved with this PR. In particular, consider:

What's changed? Why were these changes made?
- Created a new check called Ulimit to display the users current resource limits.
How should the reviewer approach this PR, especially if manual tests are required?
- Ensure that check runs locally on their machine closely resembles the provided screenshot.
Are there relevant screenshots you can add to the PR description? Expected Output (numbers don't have to match):

CyberArk internal issue ID: CNJR-772

At least 1 todo must be completed in the sections below for the PR to be merged.

[x] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update

[x] This PR includes new unit and integration tests to go with the code changes, or
[ ] The changes in this PR do not require tests

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]
[x] This PR does not require updating any documentation

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

infamousjoeg commented 1 year ago

I have faith in you, @ChrisR2487, you can bend CodeClimate to your whim! 💪🏻

imheresamir commented 1 year ago

@ChrisR2487 commits just need to be squashed down, but otherwise LGTM!

codeclimate[bot] commented 1 year ago

Code Climate has analyzed commit 5b632749 and detected 0 issues on this pull request.

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

This pull request will bring the total coverage in the repository to 94.2% (-0.3% change).

View more on Code Climate.

cyberark / conjur-inspect