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.

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

Add Macdown as an exception to minimal-socket-client-macos #340

Merged

Conversation

jedsalazar
Copy link
Contributor

Creating an exception for the Macdown application. Let me know if this isn't the correct spot for an exception

Signed-off-by: Jed Salazar <jedsalazar@gmail.com>
@jedsalazar
Copy link
Contributor Author

@tstromberg thanks for the LGTM! Looks like the verify/build tests were canceled, maybe due to a timeout?

@jedsalazar
Copy link
Contributor Author

Re-kicked the test and it timed out again. Looks like previous tests have failed for PRs that merged. I'm going to merge but let me know if I need to fix anything.

@jedsalazar jedsalazar merged commit 3914fa7 into chainguard-dev:main Dec 26, 2023
1 of 2 checks passed
@jedsalazar jedsalazar deleted the pr/jed/add-macdown-exception branch December 26, 2023 19:49
@tstromberg
Copy link
Contributor

Yeah, the addition of YARA rules appear to have caused our CI to timeout. We need to find the rules and disable them in our test rule.

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.

2 participants