Skip to content

Review Branch 2 - #7

Open
mantinone wants to merge 42 commits into
empty2from
master
Open

Review Branch 2#7
mantinone wants to merge 42 commits into
empty2from
master

Conversation

@mantinone

Copy link
Copy Markdown
Collaborator

We went a little nuts this week! We created a networked multiplayer game.

We set our own specs from scratch and tackled a difficult project without knowing ahead of time how difficult it would be! It's definitely not complete and is low on our general quality standards, but it was fun and we learned a lot. See the Readme for instructions on how to play the game networked with other people (or find Max, Ryan, Michael, or Hannah and ask us)

@johnwarejunior johnwarejunior left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code is well organized and structured.

imsolost and others added 3 commits January 5, 2018 14:36
Bumps [knex](https://github.com/tgriesser/knex) from 0.12.9 to 0.19.5.
- [Release notes](https://github.com/tgriesser/knex/releases)
- [Changelog](https://github.com/knex/knex/blob/master/CHANGELOG.md)
- [Commits](knex/knex@0.12.9...0.19.5)

Signed-off-by: dependabot[bot] <support@github.com>
…x-0.19.5

Bump knex from 0.12.9 to 0.19.5 in /empty
@lgtm-com

lgtm-com Bot commented Oct 22, 2019

Copy link
Copy Markdown

This pull request introduces 13 alerts when merging 0f37697 into afcff37 - view on LGTM.com

new alerts:

  • 5 for Unused variable, import, function or class
  • 2 for Missing variable declaration
  • 2 for Missing CSRF middleware
  • 2 for Insecure randomness
  • 1 for Unvalidated dynamic method call
  • 1 for Hard-coded credentials

dependabot Bot and others added 4 commits November 2, 2019 14:41
Bumps [morgan](https://github.com/expressjs/morgan) from 1.7.0 to 1.9.1.
- [Release notes](https://github.com/expressjs/morgan/releases)
- [Changelog](https://github.com/expressjs/morgan/blob/master/HISTORY.md)
- [Commits](expressjs/morgan@1.7.0...1.9.1)

Signed-off-by: dependabot[bot] <support@github.com>
…gan-1.9.1

Bump morgan from 1.7.0 to 1.9.1 in /empty
Bumps [debug](https://github.com/visionmedia/debug) from 2.2.0 to 4.1.1.
- [Release notes](https://github.com/visionmedia/debug/releases)
- [Commits](debug-js/debug@2.2.0...4.1.1)

Signed-off-by: dependabot[bot] <support@github.com>
…bug-4.1.1

Bump debug from 2.2.0 to 4.1.1 in /empty
@lgtm-com

lgtm-com Bot commented Nov 4, 2019

Copy link
Copy Markdown

This pull request introduces 13 alerts when merging af66604 into afcff37 - view on LGTM.com

new alerts:

  • 5 for Unused variable, import, function or class
  • 2 for Missing variable declaration
  • 2 for Missing CSRF middleware
  • 2 for Insecure randomness
  • 1 for Unvalidated dynamic method call
  • 1 for Hard-coded credentials

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.

3 participants