Store version on delete in all cases

[bosschaert]

Fixes: #85

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[bosschaert]

Note that I need to work on the tests.

[codecov]

Codecov Report

Attention: Patch coverage is 90.47619% with 4 lines in your changes missing coverage. Please review.

Project coverage is 53.64%. Comparing base (e4b15cf) to head (aed0471).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/storage/version/put.js	83.33%	2 Missing ⚠️
src/storage/object/delete.js	96.15%	1 Missing ⚠️
src/storage/object/move.js	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #86      +/-   ##
==========================================
+ Coverage   52.89%   53.64%   +0.75%     
==========================================
  Files          38       38              
  Lines        1779     1795      +16     
==========================================
+ Hits          941      963      +22     
+ Misses        838      832       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[karlpauls]

LGTM - @auniverseaway, wdyt?

[auniverseaway]

Looks good to me. It would be nice to get some performance numbers... if you delete 1000 pages, you will be creating 1000 versions.