-
Notifications
You must be signed in to change notification settings - Fork 11
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 clamav Windows Support #55
Conversation
Hi @kieranjol - thanks for the contribution! It looks like there's a missing closing |
Apologies, great catch! I must not have tested with the large files
scenario. Also thanks for merging, we are using Brunnhilde quite a lot now
in the National Library of Ireland!
…On Sat 4 Dec 2021 at 19:53, Tessa Walsh ***@***.***> wrote:
Merged #55 <#55> into develop.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#55 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAITFPXUTWRPDMC2SJSL6ZLUPJWT3ANCNFSM5HXZ6SWQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Eek, I also realise that I selected the develop branch, rather than main! Should that be merged into main? |
No that's okay! I switched it to develop :) I like to merge the whole branch into main at release time. Thank you!!
…On Mon, Dec 6, 2021, at 10:25 AM, kieranjol wrote:
Eek, I also realise that I selected the develop branch, rather than main! Should that be merged into main?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub <#55 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABTSDFA543YU273QT6QP5FDUPTIVXANCNFSM5HXZ6SWQ>.
Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Ah perfect, I just thought I'd made another mistake. Thank you Tessa! |
- Add clamav Windows Support - Add missing single quote
Hi Tessa,
So I finally got around to testing this on both Windows and MacOS - it seems to work just fine. on MacOS, the report is also printed to the terminal and to the logfile. Let me know what you think and if I need to do or test anything else.
K