Skip to content

Conversation

@csarven
Copy link
Member

@csarven csarven commented Nov 6, 2020

Copy link
Member

@acoburn acoburn left a comment

Choose a reason for hiding this comment

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

This is a very useful clarification

Copy link
Member

@dmitrizagidulin dmitrizagidulin left a comment

Choose a reason for hiding this comment

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

👍
This is great.

Copy link
Contributor

@RubenVerborgh RubenVerborgh left a comment

Choose a reason for hiding this comment

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

All good, just suggestions 👍

@csarven csarven requested a review from RubenVerborgh November 8, 2020 22:04
@csarven csarven merged commit c1a8642 into master Nov 8, 2020
@csarven
Copy link
Member Author

csarven commented Nov 8, 2020

Thanks Ruben!

Noting here that I've also discussed some of these requirements with @timbl and it was agreed - although not the exact wording in this PR. Happy to revise.

@csarven csarven deleted the feature/storage branch December 4, 2020 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants