Skip to content

Part one answer#1

Draft
orbitalbitmap wants to merge 9 commits intomasterfrom
part-one-answer
Draft

Part one answer#1
orbitalbitmap wants to merge 9 commits intomasterfrom
part-one-answer

Conversation

@orbitalbitmap
Copy link
Owner

How did I forget to make this. 🤦

@orbitalbitmap orbitalbitmap requested a review from jlcarmic May 1, 2020 03:52
Copy link
Collaborator

@jlcarmic jlcarmic left a comment

Choose a reason for hiding this comment

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

Nice job on this. As we talk about database relationships this week think about how you would add dimensionality to this, it feels flat but doesn't have to be.

Score: 100%

@@ -0,0 +1,19 @@
div(class="left-container")
Copy link
Collaborator

Choose a reason for hiding this comment

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

This design looks familiar 😂

"scripts": {
"dev": "./node_modules/.bin/nodemon index.js",
"lint": "run lint:pug-lint && run lint:eslint",
"lint": "run lint:puglint && run lint:eslint",
Copy link
Collaborator

Choose a reason for hiding this comment

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

"lint": "npm run lint:puglint && npm run lint:eslint", Someone didn't use this command :trollface:

@orbitalbitmap orbitalbitmap marked this pull request as draft May 13, 2020 01:21
@orbitalbitmap orbitalbitmap marked this pull request as ready for review May 17, 2020 17:31
@orbitalbitmap orbitalbitmap marked this pull request as draft May 18, 2020 16:41
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.

2 participants