-
Notifications
You must be signed in to change notification settings - Fork 8
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
team2-master root directory #78
Labels
Comments
@hdrdavies You already have .DS_Store in your .gitignore, but worth getting your debug log in there too. |
@iteles it's actually the other team's branch but agree with you 👍 |
Ah yes, code reviews 👍 Good on you for actually doing it! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Quite a lot of files in the root directory:
-.DS-Store and npm-debug-log can be deleted
-favicon could be in an "assets" folder
-header and footer could be in a "partials" folder
-styling could be included in another folder too if there is to be more extensive styling
The text was updated successfully, but these errors were encountered: