This repository has been archived by the owner on Feb 8, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 96
OKRs - 2019 Q4 - Package Managers #1041
Merged
Changes from all commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
332314b
okr: add link to package managers q3 okrs
Stebalien ff76842
okr: add draft pm q4 OKRs
Stebalien e53fb09
move alternatives
momack2 b0d7b8c
okrs: adjust some package manager todos
Stebalien c41249b
okrs: make the package manager OKRs concrete
Stebalien a0fcc91
okrs: npm -> guix
Stebalien File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think I buy this. DNSLink is insufficient if users care about 1) Update speed 2) Offline operation or 3) Third party persistence (i.e. not relying on the publisher to continue to maintain their package)
Ditching IPNS here means that any demo we do has to focus on IPFS being more performant/efficient then the current system which means putting all our eggs in the Bitswap sessions basket. Going with the IPNS basics (which at this point is really features like recording saved IPNS records and allowing them to be republished, all API things that do not effect the network) allows us to demo offline usage which is currently not really an option for existing package managers.
While things like Bitswap performance are almost certainly more important for package managers than IPNS support, I think offline usage makes a compelling user story even if we end up having trouble optimizing Bitswap sessions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The goal is table-stakes.
At this point, we're not shooting for a compelling user story. We're trying to make IPFS viable. At the moment.
Given that and our resource constraints, we have to make a choice. We can't choose implementing a compelling user story over fixing an issue that prevents users from even considering IPFS.