Skip to content

remove existing folders fonts and vendor from local installations #3805

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

Jump to bottom
Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

remove existing folders fonts and vendor from local installations #3805

wants to merge 1 commit into from

Conversation

khassel
Copy link
Collaborator

@khassel khassel commented Jun 1, 2025

see discussions here.

The used command should work under windows too. I'm open to better solutions.

@khassel khassel mentioned this pull request Jun 1, 2025
Merged
@sdetweil
Copy link
Collaborator

sdetweil commented Jun 1, 2025

why do you need a postinstall?

just remove folders
and git rm folders

they should be gone after pull

KristjanESPERANTO
KristjanESPERANTO approved these changes Jun 1, 2025
View reviewed changes
Copy link
Collaborator

@KristjanESPERANTO KristjanESPERANTO left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I just successfully tested it on my system (Debian 12) 👍

@khassel
Copy link
Collaborator Author

khassel commented Jun 1, 2025

they should be gone after pull

all files under git control are gone but not the ignored files in node_modules. You can test it locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KristjanESPERANTO KristjanESPERANTO KristjanESPERANTO approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@khassel @sdetweil @KristjanESPERANTO