codepath / codepath/flixster-starter

Feedback on week2

Open
#61 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
JavaScript
Stars
0
Forks
225
PR merge metrics
No merged PRs in 30d

Description

Overall I saw great improvement and growth from week 2 project which is very good. I can see quite amount of improvement from our last 1-1
## Code Structure
From week 1, the code is more structured and put into reasonable directories.
Still, some unused .css files end up being pushed into the final repo. They should be excluded.
## Style
Pretty clean and tight.
Nit: Some inconsistent whitespacing, and code can be cleaner with certain choices. This is really not a convenient place to extend, will cover them during 1-1

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.