-
Notifications
You must be signed in to change notification settings - Fork 34
yeet gulp from build system (involves lots of chaos) #445
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
Conversation
stevenmirabito
left a comment
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.
Impressive work, this looks great.
520a85d to
d5ef6a9
Compare
gonna crash out over env variables stuff I'm so bad at updating the prod one when I change local stuff
costowell
left a comment
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.
Nevermind this is not a you problem, it's a problem from elsewhere |
|
#280 :) |
costowell
left a comment
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.
LGTM


Removed gulp from the build system for the frontend. I'd like to add back in the linting for frontend that happened but also that wasn't really working so it's not a regression either.
Also I'm sorry some of it is insane trying to update the build system with 8-9 year old dependencies in the project isn't fun (soon tm)