Skip to content

Add a test + fix for #19#20

Open
Civil wants to merge 2 commits intojjneely:masterfrom
Civil:fill-bug
Open

Add a test + fix for #19#20
Civil wants to merge 2 commits intojjneely:masterfrom
Civil:fill-bug

Conversation

@Civil
Copy link

@Civil Civil commented Aug 14, 2017

This is only adding a test for #19

@Civil Civil changed the title Add a test for #4 Add a test for #19 Aug 14, 2017
@Civil Civil force-pushed the fill-bug branch 2 times, most recently from 1e09504 to 7cf79e9 Compare August 15, 2017 08:10
Vladimir Smirnov added 2 commits August 15, 2017 10:10
This is only adding a test for jjneely#19
…e it

Fill must read and write data to exactly same archive.

For this purpose this commit intoroduce new function 'UpdateManyWithRetention'
that also gets 'desired' archive's retention.

Also migrate 'fillArchive' in fill package to use that function, instead of
plain 'UpdateMany'

Fixes jjneely#19
Civil pushed a commit to Civil/buckytools that referenced this pull request Oct 18, 2017
@Civil Civil changed the title Add a test for #19 Add a text + fix for #19 Nov 7, 2017
@Civil Civil changed the title Add a text + fix for #19 Add a test + fix for #19 Nov 7, 2017
azhiltsov pushed a commit to azhiltsov/buckytools that referenced this pull request Dec 1, 2017
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.

1 participant