Skip to content

Remove source folder after build #1012

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.

Sign up for GitHub

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

Draft
Jelle-S wants to merge 1 commit into
base: 6.x
Choose a base branch
from
Draft

Remove source folder after build #1012

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions scripts/install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 •
edited
Loading

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

@Jelle-S Jelle-S Dec 10, 2024

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...

Copy link
Contributor

@SvenVdv SvenVdv Apr 29, 2025

Choose a reason for hiding this comment

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

@Jelle-S Is het mss mogelijk om die source folder uit te sluiten van een build, bv. via .npmignore (voor NPM builds) of in de CI/CD excluden?