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

Removed unused examples / unused tests / unused diagrams #354

Merged
merged 2 commits into from
Mar 5, 2025

Conversation

jogu
Copy link
Collaborator

@jogu jogu commented Nov 30, 2024

These are all essentially unmaintained and likely incorrect, so remove them to avoid misleading anyone. e.g. the schema is out of date and the tests hence don't pass for the current examples.

Anyone who thinks any of these are important is welcome to add them back in a separate PR, bring them up to date and incorporate them in the spec / GitHub actions as appropriate.

closes #206 (by making it irrelevant)

These are all essentially unmaintained and likely incorrect, so remove
them to avoid misleading anyone.

Anyone who thinks any of these are important is welcome to add them back,
bring them up to date and incorporate them in the spec / GitHub actions
as appropriate.

closes #206 (by making it irrelevant)
@jogu jogu added the editorial label Nov 30, 2024
@@ -4,27 +4,6 @@

The current WG-Draft version is built automatically from the master branch and can be accessed at: https://openid.github.io/OpenID4VP/openid-4-verifiable-presentations-wg-draft.html

### Running Tests ###
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am not ok removing these without adding a URL or something to the conformance tests or something equivalent.

Copy link
Collaborator Author

@jogu jogu Dec 9, 2024

Choose a reason for hiding this comment

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

I'm happy to add a link to conformance tests and have done so, thanks!

Just to clarify (and I've updated the main description of the PR to say this now), these tests test the examples in the specification against a schema that's not up to date, so the tests fail if you try to run them. Keeping an incorrect & unmaintained schema in the repo is just very unhelpful to anyone looking at the repo.

I'm making no judgement on the value of these things and would be very happy if someone wants to raise a new PR that puts back / updates the schema and adds running the tests into the GitHub actions.

Copy link
Collaborator

@Sakurann Sakurann left a comment

Choose a reason for hiding this comment

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

still not fully convinced but yeah not gonna block

@jogu jogu merged commit 04ec961 into main Mar 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorporate browser API sequence diagrams into the specification
3 participants