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

Paper Cleanups #102

Merged
merged 6 commits into from
Jan 12, 2018
Merged

Conversation

bnewbold
Copy link
Collaborator

The intent of this PR is to put the whitepaper(s) in a reasonable state (no TODOs or partial documentation of mutli-writer) while we shift efforts to the DEP process.

It might be worthwhile to do one more round of small additions after this PR such that the paper correctly documents the 2017 dat protocol behavior (eg, the whitepaper doesn't mention a couple crucial details like keep-alive packets or how to delete hyperdrive entries).

r? @mafintosh @Karissa (as paper co-authors)

This file had no content and was confusing.
Consensus is to document multi-writer via a DEP (RFC-like) process
first, and not propose new changes in the whitepaper first.
This could probably be clarified better (minor revisions and bug-fixes
vs. substantial updates), and maybe we want to pull out the original
"1.0" paper for historical reference. Let's wait until DEP process
settles down first.
@joehand
Copy link
Collaborator

joehand commented Jan 12, 2018

All looks good! Thanks.

@joehand joehand merged commit 9f0ce82 into dat-ecosystem-archive:master Jan 12, 2018
@joehand
Copy link
Collaborator

joehand commented Jan 12, 2018

@bnewbold I moved the whitepaper over https://github.com/datprotocol/whitepaper.

Where there any other changes you had in mind to marked the paper as "frozen"? If it all looks good, I'll delete the paper stuff here.

@bnewbold
Copy link
Collaborator Author

I think we can delete the paper here, i'll add any future notes to the docs or a DEP.

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.

2 participants