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

schemadsl: tests operating from testmark formatted fixtures. #381

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

Conversation

warpfork
Copy link
Collaborator

@warpfork warpfork commented Mar 7, 2022

This is covering mostly similar ground to the previous fixtures in yaml. The pointer to the ipld/ipld repo advances to track the ipld/ipld#144 branch over there, which is the conversion process for the fixtures. That PR also describes the impetus.

This is covering mostly similar ground to the previous fixtures
in yaml.  The reasons we want to move this way are:
1) a testmark parser dependency is a bit lighter than a yaml parser;
2) much more importantly, the testmark formatted fixtures have
literate documentation and explanation interspersed, which
is valuable for making the fixtures do double duty as
valuable documentation and legible examples.
@warpfork warpfork marked this pull request as draft March 7, 2022 16:01
@rvagg rvagg self-assigned this Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🥞 Todo
Development

Successfully merging this pull request may close these issues.

2 participants