-
Notifications
You must be signed in to change notification settings - Fork 24
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
Rework directory structure #114
Conversation
- deleted go backend - hoisted env files - added debugging - renamed frontend folder - renamed backend folder
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes are mostly related to:
- renaming files
- removing Tremor
- Running prettier/eslint
Change LGTM! I'll test Codespaces once this merges just to make sure we're still in good shape there 👍
@@ -1,17 +0,0 @@ | |||
{ | |||
"typescript.enablePromptUseWorkspaceTsdk": true, | |||
"typescript.format.enable": true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't remember, was this replaced by prettier config?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes :)
@@ -0,0 +1,232 @@ | |||
/* eslint-disable import/no-extraneous-dependencies */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this file from a template? If so, might be helpful to link to it for reference?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was here when I joined the project but has lots of github related rules
"settings": {} | ||
}, | ||
"extensions": [ | ||
"github.copilot", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we'll need to test this after merging, as I'm not sure how to create a Codespace using the file from this branch. I think it should work though.
FWIW Co-pilot was not installed in my new Codespace by default, but golang.go
was, so I'm not sure what happened there 🤷
This does a lot so hold onto your 🎩s
backend
andapp
respectivelystart
todev