Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(random): Adds Random component to generate deterministic random values #63

Merged
merged 11 commits into from
Nov 14, 2018
Merged

feat(random): Adds Random component to generate deterministic random values #63

merged 11 commits into from
Nov 14, 2018

Conversation

jcowman2
Copy link
Contributor

Description

  • Adds InstanceRandom interface and implementation to generate deterministic, pseudo-random data from a GameInstance.
  • Adds seed config option which can be used to specify a seed from which all random data is generated. If no seed is specified, one will be generated and persisted.
  • 100% unit test coverage and documentation

Related Issues

@jcowman2 jcowman2 added this to the 0.5.0 milestone Nov 14, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 188

  • 67 of 67 (100.0%) changed or added relevant lines in 9 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%
Totals Coverage Status
Change from base Build 183: 0.0%
Covered Lines: 862
Relevant Lines: 862
💛 - Coveralls

@jcowman2 jcowman2 merged commit 5398c53 into 0.x Nov 14, 2018
@jcowman2 jcowman2 deleted the feat/52-random branch November 14, 2018 06:16
@jcowman2 jcowman2 mentioned this pull request Nov 14, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@jcowman2 @coveralls