Skip to content
This repository was archived by the owner on Jun 24, 2026. It is now read-only.

Add doc, fix output bugs#1769

Merged
ehennenfent merged 3 commits into
masterfrom
dev-verifier-doc
Jul 9, 2020
Merged

Add doc, fix output bugs#1769
ehennenfent merged 3 commits into
masterfrom
dev-verifier-doc

Conversation

@feliam

@feliam feliam commented Jul 6, 2020

Copy link
Copy Markdown
Contributor

Readme updated to link to manticore-verifier documentation
Typos and output cleanups

@feliam
feliam requested a review from ehennenfent July 6, 2020 15:46
Comment thread docs/index.rst
:maxdepth: 2
:caption: Contents:

verifier

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This may need to be CLI-tools
And then inside that we should have manpage-like documentation on how to use the commandline tools for each tool. (So far we have plain manticore and manticore-verifier )

@ehennenfent ehennenfent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR! I suggested a few spelling fixes, but overall the docs looks like a pretty comprehensive introduction to the verifier. If we intend for this to be used externally, I think it'd also be worth adding a section to the README with a sentence or two about manticore-verifier and a link to the documentation on RTD.

Comment thread manticore/ethereum/verifier.py Outdated
Comment thread manticore/ethereum/verifier.py Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
Comment thread docs/verifier.rst Outdated
feliam and others added 2 commits July 6, 2020 20:38
Co-authored-by: Eric Hennenfent <eric.hennenfent@trailofbits.com>
@feliam
feliam requested a review from ehennenfent July 7, 2020 19:31

@ehennenfent ehennenfent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! Thanks for adding the README link.

@ehennenfent
ehennenfent merged commit da51351 into master Jul 9, 2020
@ehennenfent
ehennenfent deleted the dev-verifier-doc branch July 9, 2020 19:55
ekilmer added a commit that referenced this pull request Jul 17, 2020
* master:
  Create a model for strncpy (#1770)
  Add doc, fix output bugs (#1769)
  Update EVM usage example (#1772)
  New working model of strlen (#1725)
  Typo (#1768)
  Specialized iterative serialization for Array (#1756)
  Enable nightly uploads to PyPI (#1757)
  Manticore 0.3.4 (#1720)
  Manticore verifier (#1717)
  Nightly MacOS Tests (#1614)
  Remove/procrastinate solver query in ether leak detector (#1727)
  Fix constant folding & constraint set slicing (#1706)
ekilmer added a commit that referenced this pull request Aug 5, 2020
* master:
  Change the default to threading (#1779)
  Coveralls Take 2 (#1784)
  Capture return values in run.sh (#1776)
  Create a model for strncpy (#1770)
  Add doc, fix output bugs (#1769)
@ehennenfent ehennenfent added this to the Manticore 0.3.5 milestone Sep 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants