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

FRC: Piece Multihash CID #758

Merged

Conversation

aschmahmann
Copy link
Contributor

@aschmahmann aschmahmann commented Jul 26, 2023

Defines an alternative CID representation for Filecoin Piece Commitments (i.e. CommP)

Rendered Version

Discussion in #759

@@ -0,0 +1,84 @@
| fip | title | author | discussions-to | status | type | created |
| --- | --- | --- | --- | --- | --- | --- |
| TBD | Piece Multihash | Adin Schmahmann (@aschmahmann), Peter Rabbitson (@ribasushi), ... | https://github.com/filecoin-project/FIPs/discussions/759 | Draft | FRC | 2023-MM-DD |
Copy link
Member

Choose a reason for hiding this comment

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

Please follow the vertical table format from the template. The FIP number and author list need to be quoted.

Copy link
Contributor Author

@aschmahmann aschmahmann Jul 31, 2023

Choose a reason for hiding this comment

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

Thanks @anorth. I just copied the format from #709, but I see that got fixed up in #762. Should be resolved now. Let me know if there are any other procedural issues to correct here (or you have any thoughts on the FRC itself 😄).

For example, when are FRC numbers assigned and is the creation date when the PR is opened or the day it's merged?

Copy link
Member

Choose a reason for hiding this comment

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

Everything else looks fine from an editorial standpoint. I will reserve discussion of the content for the discussion forum, but don't think I have much to add to what's already there.

The date can be either, the FIP number will be assigned and filled in by an editor.

Defines an alternative CID representation for Filecoin Piece Commitments
(i.e. CommP)
Copy link
Collaborator

@kaitlin-beegle kaitlin-beegle left a comment

Choose a reason for hiding this comment

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

Approved for editorial changes. As an FRC, this document should be ready for merging.

@kaitlin-beegle kaitlin-beegle merged commit 18491e9 into filecoin-project:master Aug 8, 2023
3 checks passed
@aschmahmann
Copy link
Contributor Author

@kaitlin-beegle I'm still hoping to see some conversations around this FIP resolved before finalizing it. Is it ok to make PRs to this FRC document (0069) or are changes expected to be per-FRC number?

@anorth
Copy link
Member

anorth commented Aug 9, 2023

Yes, it's totally ok to make PRs to this FRC document.

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.

None yet

3 participants