The size of the 8.x-1.0-beta7 binary is 156.11 KB, and this is due to a 1.5MB (when expanded) database dump being distributed along with the module. The database dump adds no functionality and might deter people from downloading the module.

If the database dump needs to live in the git repo for different developers to make use of it, I suggest it's removed from the branch before any release is tagged, and then if it's still needed for development, the commit which removes the file can be reverted after a tag has been released, bringing the file back.

Comments

svenryen created an issue. See original summary.

killua99’s picture

killua99 commented

the dump is require for test. I'll take a look how we could get rid of that database, perhaps using unit will change that.

svenryen’s picture

svenryen commented

It seems to me that the test is for a one-time change that took place in beta6/beta7, so it's a valid question whether this test is really required any more now that the feature is implemented and has passed testing? It's quite obvious to me that nobody would edit *.permissions.yml and reintroduce English title case for the permission names.

svenryen’s picture

svenryen commented

We removed the database dump in this issue.
https://www.drupal.org/project/eu_cookie_compliance/issues/2900209

svenryen’s picture

svenryen commented
Status: Active » Closed (duplicate)