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
[jamf_protect] Update default.yml pipeline #9581
Conversation
/test |
🚀 Benchmarks reportTo see the full report comment with |
💚 Build Succeeded
|
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to have some system tests that would have caught this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏼
I agree with @efd6 on adding system tests.
Package jamf_protect - 0.3.0 containing this change is available at https://epr.elastic.co/search?package=jamf_protect |
Proposed commit message
Checklist
changelog.yml
file.How to test this PR locally