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 source folder after build #1012

Open
wants to merge 1 commit into
base: 6.x
Choose a base branch
from
Open

Conversation

Jelle-S
Copy link
Contributor

@Jelle-S Jelle-S commented Aug 8, 2024

After the build process is done, we have no more need for the source folder. Deleting it saves us around 150MB on the server where it is deployed, for each release we do.

After the build process is done, we have no more need for the source folder.
Deleting it saves us around 150MB on the server where it is deployed, for
each release we do.
@Jelle-S Jelle-S requested a review from lennartvava August 8, 2024 09:51
@@ -38,3 +38,6 @@ mkdir -p ../build/@digipolis-gent/modal;
cp -R ./node_modules/@digipolis-gent/modal/dist/index.js ../build/@digipolis-gent/modal;
cp -R ./node_modules/gent_styleguide/build/styleguide ../build;
./node_modules/.bin/gulp build;

echo "Removing gent_base 'source' directory...";
rm -rf ../source;
Copy link
Collaborator

@lennartvava lennartvava Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lokaal blijft het handig om de source niet te verwijderen tijdens het install script, ook bij bijv. een git checkout versie van het thema waarbij het install script gerund wordt, dan heb je uiteraard de source zeker nodig om aanpassingen te maken. Is het misschien een idee om een install-prod.sh te maken waarbij install.sh ingeladen wordt + extra commands zoals in deze commit?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dat is een idee, maar dan moeten we op één of andere manier zeker zijn dat die install-prod.sh telkens bij composer install wordt uitgevoerd wanneer deze op onze deploy server wordt uitgevoerd, want momenteel wordt deze install.sh als post-install-script van composer uitgevoerd overal waar gent_base wordt gebruikt. Als we dat aanpassen loopt dat script ook nog steeds bij jullie lokaal...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants