Skip to content

Add Armour #444

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

Merged
merged 13 commits into from
Apr 14, 2025
Merged

Add Armour #444

merged 13 commits into from
Apr 14, 2025

Conversation

rohan-stepsecurity
Copy link
Member

No description provided.

rohan-stepsecurity and others added 12 commits April 14, 2025 10:38
(cherry picked from commit 89aff7a)
(cherry picked from commit 13695da)
(cherry picked from commit 56f65a0)
(cherry picked from commit cd1cb63)
(cherry picked from commit ade2838)
(cherry picked from commit 6b615ea)
(cherry picked from commit e0dc8ee)
fix: function sequencing
(cherry picked from commit 35ee5f0)
(cherry picked from commit 6e91234)
(cherry picked from commit 731bcf0)
(cherry picked from commit 93890c0)
Copy link
Contributor

@step-security-bot step-security-bot left a comment

Choose a reason for hiding this comment

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

Code Review Skipped

StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files

Feedback

We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.

Copy link
Contributor

@step-security-bot step-security-bot left a comment

Choose a reason for hiding this comment

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

Code Review Skipped

StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files

Feedback

We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.

Copy link

codecov bot commented Apr 14, 2025

Codecov Report

Attention: Patch coverage is 15.12915% with 230 lines in your changes missing coverage. Please review.

Project coverage is 62.78%. Comparing base (0da07db) to head (1be8ea8).
Report is 14 commits behind head on main.

Files with missing lines Patch % Lines
common.go 0.00% 74 Missing ⚠️
sudo.go 0.00% 68 Missing ⚠️
agent.go 8.33% 40 Missing and 4 partials ⚠️
apiclient.go 45.83% 12 Missing and 1 partial ⚠️
dnsconfig.go 7.14% 13 Missing ⚠️
global_feature_flags.go 66.66% 10 Missing and 2 partials ⚠️
firewall.go 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #444      +/-   ##
==========================================
- Coverage   69.69%   62.78%   -6.91%     
==========================================
  Files          15       17       +2     
  Lines        1686     1932     +246     
==========================================
+ Hits         1175     1213      +38     
- Misses        392      593     +201     
- Partials      119      126       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@varunsh-coder varunsh-coder merged commit e2bfba3 into main Apr 14, 2025
5 of 7 checks passed
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.

3 participants