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.

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

refactor(game): begin refactoring into a game state and game level pa… #20

Merged
merged 1 commit into from Dec 29, 2015
Merged

refactor(game): begin refactoring into a game state and game level pa… #20

merged 1 commit into from Dec 29, 2015

Conversation

seiyria
Copy link
Collaborator

@seiyria seiyria commented Dec 29, 2015

…ttern

if (gamestate.playing) config.world.step(1/60);
config.renderer.render(config.stage);
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

EOF newlines please

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

why

@Fishrock123
Copy link
Collaborator

Could we name Class-exporting files by caps like we've been doing?

@Fishrock123
Copy link
Collaborator

also uh, commit message length

@seiyria
Copy link
Collaborator Author

seiyria commented Dec 29, 2015

Is that how they're capitalized? It seemed mostly random to me. I would prefer if either all files are capitalized or none of them are because that's a mostly-useless notation if the file structure makes sense (currently everything is just in one folder, so this will require a bit of a re-org).

Also, if we want to enforce commit message length we should probably have a bot for that (gitcop is ok).

@Fishrock123
Copy link
Collaborator

Just fix your vim or whatever you commit with so that it enforces 50colunns message, 72 columns description

@seiyria
Copy link
Collaborator Author

seiyria commented Dec 29, 2015

I use the terminal to commit, but you haven't given my a reason why I should care about commit message length. I don't do things without a sufficient reason why.

@Fishrock123
Copy link
Collaborator

It cuts off the description in git. Describe what you can in the message and then dump the rest in the description. Be concise.

Also >72 lines description wraps badly in terminals.

@seiyria
Copy link
Collaborator Author

seiyria commented Dec 29, 2015

Make your terminal larger ????

idk, I'm not rebasing or changing messages but we'll see going forward.

@Fishrock123
Copy link
Collaborator

Fine I'll just fix your commits for you when I get home.

Also git does it regardless of your terminal size.

Rabrennie added a commit that referenced this pull request Dec 29, 2015
refactor(game): begin refactoring into a game state and game level pa…
@Rabrennie Rabrennie merged commit 77070bf into Rabrennie:master Dec 29, 2015
@seiyria seiyria deleted the refactor-game-states branch December 29, 2015 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants