Skip to content

Commit

Permalink
Updated review comments, see #99
Browse files Browse the repository at this point in the history
  • Loading branch information
samreid committed Jun 15, 2018
1 parent 5a2797e commit cd319ee
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions js/game/model/GameState.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
// Copyright 2017-2018, University of Colorado Boulder

/**
* Enumeration for all states that the game can be in.
* Enumeration for all states that the game can be in. See https://github.com/phetsims/vegas/issues/74 about whether
* this can/should be factored out.
*
* @author Jonathan Olson <jonathan.olson@colorado.edu>
* REVIEW: These states and corresponding logic are duplicated across many games. Do we have the potential to factor something out?
* REVIEW*: Probably not easily, as showing a "correct" solution, giving two attempts, and having a "level complete"
* REVIEW*: state are not-at-all universal for game sims.
*/
define( function( require ) {
'use strict';
Expand Down

0 comments on commit cd319ee

Please sign in to comment.