-
Notifications
You must be signed in to change notification settings - Fork 807
485/handle missing binaries #670
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
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #670 +/- ##
========================================
Coverage 56.62% 56.62%
========================================
Files 120 120
Lines 4044 4044
========================================
Hits 2290 2290
Misses 1754 1754 Continue to review full report at Codecov.
|
<i className="glyphicon glyphicon-info-sign" style={{'marginRight': '5px'}}/> | ||
Some Monkey binaries are not found where they should be...<br/> | ||
Try downloading them from <a href="https://github.com/guardicore/monkey/releases/latest" target="blank">here</a>, | ||
at the bottommost section titled "Assets". |
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.
Ideally we should add "... and place them in monkey/monkey_island/cc/binaries
directory."
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.
I wish I could make the directory path look like that
, I used italics instead 🙄
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.
You can do this
using import marked from 'marked';
and then styling your text using Markdown. You can see an example of this in monkey_island/cc/ui/src/components/attack/techniques/MitigationsComponent.js
.
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.
Alternatively, use a div
with the className="label"
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.
What is this?
Fixes #485 .
Add any further explanations here.
Checklist
Proof that it works
If applicable, add screenshots or log transcripts of the feature working

Changes
Are the commit messages enough? If not, elaborate.