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

CIP-0016 | Adjust preamble and structure w.r.t CIP-0001 #660

Merged
merged 3 commits into from
Jan 6, 2024

Conversation

rphair
Copy link
Collaborator

@rphair rphair commented Dec 20, 2023

@rphair
Copy link
Collaborator Author

rphair commented Dec 20, 2023

@intricate although this was merged as Active I didn't see that any commercial or community implementations were indicated in the linked discussion, nor could I find any of these today.

Since the CIP as merged didn't have anything like a Path to Active indicating progress that would have validated it as Active, I've reset the status to Proposed. Perhaps it was given Active status based on expectation or as a formality.

In any case, if you or anyone else know implementations of this proposed standard then please post here and we can change the status back to Active if appropriate.

Copy link
Collaborator

@Ryun1 Ryun1 left a comment

Choose a reason for hiding this comment

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

or anyone else know implementations of this proposed standard

I think jcli, cardano-signer and CSL support these formats.

@rphair
Copy link
Collaborator Author

rphair commented Dec 31, 2023

That's why we need you @Ryun1 😎 ... resetting back to Active status with proper documentation of current implementations as you've indicated.

Also I realised in hindsight that CIP-0001 allows N/A for an Implementation Plan which I think is appropriate when real-world implementations precede or substitute a designated reference implementation... so I've also set that in this typical case.

Copy link
Collaborator

@Ryun1 Ryun1 left a comment

Choose a reason for hiding this comment

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

That's why we need you @Ryun1 😎 ... resetting back to Active status with proper documentation of current implementations as you've indicated.

This is why we are a team!

@rphair rphair requested a review from Crypto2099 December 31, 2023 21:12
Copy link
Collaborator

@Crypto2099 Crypto2099 left a comment

Choose a reason for hiding this comment

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

LGTM

@rphair rphair merged commit 9edebbe into cardano-foundation:master Jan 6, 2024
@rphair rphair removed the Remediation label Jan 7, 2024
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.

CIP-0016 | Adjust preamble and structure w.r.t CIP-0001
3 participants