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

Add validation of invariants for PublicIdType and ImmutableData, see … #62

Merged
merged 1 commit into from Jun 1, 2015
Merged

Conversation

ghost
Copy link

@ghost ghost commented Jun 1, 2015

…comment at MAID-1056. Fixes #57.

Review on Reviewable

@maidsafe-highfive
Copy link

r? @dirvine

(maidsafe_highfive has picked a reviewer for you, use r? to override)

@dirvine
Copy link
Contributor

dirvine commented Jun 1, 2015

Review status: all files reviewed, all discussions resolved, some commit checks pending.
Reviewed files:

  • src/data/immutable_data.rs @ r1
  • src/helper.rs @ r1
  • src/id/public_id_type.rs @ r1

Comments from the review on Reviewable.io

dirvine added a commit that referenced this pull request Jun 1, 2015
Add validation of invariants for PublicIdType and ImmutableData, see …
@dirvine dirvine merged commit 42a4f03 into maidsafe-archive:master Jun 1, 2015
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 92.74% when pulling 4140731 on s-brian:MAID-1056 into 4049d56 on maidsafe:master.

@ghost ghost deleted the MAID-1056 branch June 1, 2015 21:08
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.

index problem in name creation
3 participants