Skip to content
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

Remove Backend Assets from Dist Folder #140

Merged
merged 1 commit into from
Mar 15, 2022
Merged

Conversation

imRohan
Copy link
Owner

@imRohan imRohan commented Mar 15, 2022

We initially added the dist folder to the repo to speed up our deploys due to the
front end assets taking too long to compile. In doing so, we also included the
backend assets. These items can be discarded as they do not slow down deploys.

This change addresses the need by:

  • Deleting backend folders
  • Isolating the frontend assets through gitignore

We initally added the dist folder to the repo to speed up our deploys due to the
front end assets taking too long to compile. In doing so, we also included the
backend assets. These items can be discarded as they do not slow down deploys.

This change addresses the need by:
* Deleting backend folders
* Isolating the frontend assets through gitignore
@imRohan imRohan added the Ready for Review Once you've squashed your commits, the PR is now ready for review! label Mar 15, 2022
@imRohan imRohan self-assigned this Mar 15, 2022
@imRohan imRohan merged commit 222ce6e into master Mar 15, 2022
@imRohan imRohan deleted the rl-remove-dist-folders branch March 15, 2022 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for Review Once you've squashed your commits, the PR is now ready for review!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant