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

Add fix to prevent accumulation of dist.zip content #300

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

matthme
Copy link
Collaborator

@matthme matthme commented Jun 3, 2024

I should have added this a long time ago. It turns out that the dist.zip file keeps accumulating in size with each npm run package step if it is not being removed before packaging (presumably because bestzip does not overwrite the dist.zip file and if files in the dist folder get renamed they are simply added to the archive). That way webhapps can grow to humongous sizes over time without the developer being aware of it.

@matthme matthme requested a review from c12i June 3, 2024 21:21
@c12i c12i added the ShouldBackport/0.3 This change should be backported to develop-0.3 label Jun 4, 2024
@c12i c12i merged commit c2fe750 into develop Jun 4, 2024
2 of 7 checks passed
@c12i c12i deleted the fix/rimraf-dist branch June 4, 2024 08:48
c12i pushed a commit to c12i/scaffolding that referenced this pull request Jun 4, 2024
@c12i c12i added ShouldBackport/0.2 This change should be backported to develop-0.2 and removed ShouldBackport/0.2 This change should be backported to develop-0.2 labels Jun 5, 2024
c12i pushed a commit to c12i/scaffolding that referenced this pull request Jun 5, 2024
c12i added a commit that referenced this pull request Jun 6, 2024
* fix: poorly formatted rust code (#288)

* fix: poorly formatted code in coordinator zome

* document add_newlines

* clearn up coordinators module

* clean up integrity module

* refactor utils

* fix add_newlines

* update integrity comments

* fix map file calls

* fix add_newlines

* make add_newlines public/private function agnostic

* refactor link-type coordinator

* fix  remove link handler

* fix collection coordinator

* clean up entry-type coordinator

* fix clippy warnings

* refactor link-type coordinator

* fix bug scaffolding entry-type with linked_from

* fix target_hash_variable

* remove unnecessary format usage

* remove unnecessary clone

* fix validate_delete_result

* move comment

* fix entry from rcord error message

* rename unparse function to unparse_pretty;

* feat: add long disable fast track option to hc scaffold web-app (#293)

* add fix to prevent accumulation of dist.zip content (#300)

* chore: bump version

---------

Co-authored-by: matthme <36768177+matthme@users.noreply.github.com>
@c12i c12i removed the ShouldBackport/0.3 This change should be backported to develop-0.3 label Jun 19, 2024
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.

None yet

2 participants