Inline examples
The current method of putting examples in a different directory makes it very difficult to read the .md text and examples and check that they are both consistent with each other. Can we ask the editor to copy all the examples into the body of the text to enable proof reading to be completed more easily.
Comments (8)
-
-
there’s an incorrect filename in the ‘presentations’ README.md as well
-
reporter @Mark I tried the docker command you gave me and got a no such file error
davidchadwick@DavidsMBPRetina connect % docker run -v
pwd
:/data danielfett/markdown2rfc openid-4-verifiable-credential-issuance-1_0.md
Unable to find image 'danielfett/markdown2rfc:latest' locally
latest: Pulling from danielfett/markdown2rfc
68ced04f60ab: Pull complete
d1d516f83cca: Pull complete
1c69061f55c9: Pull complete
cb02ba23f0ac: Pull complete
b1ed9df08170: Pull complete
0b9fc8520bb7: Pull complete
11af64e82fbb: Pull complete
a1283f980933: Pull complete
b4503901ce48: Pull complete
97251e9a8ad2: Pull complete
Digest: sha256:a107b0a45899a34bb7928ca2879dc6c4743f9b1b822ba559bba707b6fa663bf5
Status: Downloaded newer image for danielfett/markdown2rfc:latest
grep: openid-4-verifiable-credential-issuance-1_0.md: No such file or directory
2022/07/22 15:18:47 Couldn't open "openid-4-verifiable-credential-issuance-1_0.md": "open openid-4-verifiable-credential-issuance-1_0.md: no such file or directory"
Error: Unable to parse the XML document: ./.xml
davidchadwick@DavidsMBPRetina connect % -
There is now a pipeline that compiles the markdown file into an html file with every commit done for an unmerged PR.
To check out an html for a PR you are trying to review,- please go to the PR and click pipelines on the right hand side.
2. Go to Artifacts section (like here) and click html, it will download a tar.gz file which you will have to unzip, but after that you will have an html :)
-
-
assigned issue to
-
assigned issue to
-
- changed status to open
pending close - will close in a week if no major issues/questions with the new mechanisms that generates html files that can be used to easily review the PRs.
will probably try update the script to publish an html to openid.bitbucket.io/unmerged-branch-name/openid-specname.html, so that reviewers don't have to download html files, but probably after the rounds of the implementers drafts :)
-
reporter Thankyou Kristina. This is very helpful (especially for PRs with many changes).
-
- changed status to resolved
- Log in to comment
Should be able to clone and build the html on demand using the docker command in the README.md (at least for credential issuance) Unfortunately the command in the README has an incorrect filename. The command should be…
It might also be quite useful to have the each snapshot auto build for those members of the group who don’t have a docker environment to work with. The eKYC&IDA repo has that already operating… https://bitbucket.org/openid/ekyc-ida/deployments