Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Argument parsing from CLI #204

Merged
merged 4 commits into from Jan 24, 2016
Merged

Argument parsing from CLI #204

merged 4 commits into from Jan 24, 2016

Conversation

gavofyork
Copy link
Contributor

No description provided.

@gavofyork gavofyork added the A0-pleasereview 🤓 Pull request needs code review. label Jan 23, 2016
@@ -91,6 +91,8 @@ extern crate evmjit;
#[macro_use]
extern crate ethcore_util as util;

// NOTE: Add doc parser exception for these pub declarations.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should document public modules instead. Remove TODOs from here and add top level documentation to each public module file. There should be no warnings after this

@arkpar arkpar added A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 24, 2016
arkpar added a commit that referenced this pull request Jan 24, 2016
Argument parsing from CLI
@arkpar arkpar merged commit 0e296e7 into master Jan 24, 2016
@gavofyork gavofyork deleted the gav branch January 25, 2016 09:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants