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-0108 | Ensure "CIP-0008" is in capital letters #951

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

palas
Copy link

@palas palas commented Dec 16, 2024

This PR addresses the issue #949, by ensuring CIP-0008 is used in capital letters in the witnessAlgorithm field of the witness type in CIP-0108, updating the "no-confidence" example and the test vector Markdown documentation.

Fixes #949.

@palas palas changed the title Ensure "CIP-0008" is in capital letters (address #949) CIP-0108 | Ensure "CIP-0008" is in capital letters (address #949) Dec 16, 2024
@rphair rphair changed the title CIP-0108 | Ensure "CIP-0008" is in capital letters (address #949) CIP-0108 | Ensure "CIP-0008" is in capital letters Dec 16, 2024
@rphair rphair added the Category: Metadata Proposals belonging to the 'Metadata' category. label Dec 16, 2024
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

Can't think of any technical things that would fail because of this schema change but please @Ryun1 @Crypto2099 @perturbing @gitmachtl confirm if you can.

@rphair rphair added Update Adds content or significantly reworks an existing proposal State: Last Check Review favourable with disputes resolved; staged for merging. labels Dec 16, 2024
@Ryun1
Copy link
Collaborator

Ryun1 commented Dec 18, 2024

Great spot @palas
ill reach out to tooling to double check that this change wouldnt break their implementations before we merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Metadata Proposals belonging to the 'Metadata' category. State: Last Check Review favourable with disputes resolved; staged for merging. Update Adds content or significantly reworks an existing proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CIP-0108 - Inconsistent field name
3 participants