Skip to content

Add support for Microdata#343

Open
toastal wants to merge 5 commits into
ocsigen:masterfrom
toastal:microdata
Open

Add support for Microdata#343
toastal wants to merge 5 commits into
ocsigen:masterfrom
toastal:microdata

Conversation

@toastal

@toastal toastal commented May 13, 2025

Copy link
Copy Markdown
Contributor

See: https://html.spec.whatwg.org/multipage/microdata.html

1

Footnotes

  1. Please consider giving up MS GitHub or offering a non-proprietary, non-US-corporate-controlled mirror for this free software project. I wish to delete this Microsoft account in the future, but I need more projects like this to support alternative methods to send patches & contribute.

@toastal

toastal commented May 26, 2025

Copy link
Copy Markdown
Contributor Author

These build errors also happen on the default branch. Just did a fresh checkout to find:

File "ppx/tyxml_ppx.ml", line 381, characters 18-65:
381 |   | Pexp_function (params, constraint_, (Pfunction_body content)) ->
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Error: This pattern matches values of type 'a * 'b * 'c
       but a pattern was expected which matches values of type case list
Generating: index.html  

I have been using this branch in a Nix overlay for 2 weeks now.

@toastal

toastal commented May 26, 2025

Copy link
Copy Markdown
Contributor Author

It appears to me dune test fails starting at 7b11831 from #340

@toastal

toastal commented May 27, 2025

Copy link
Copy Markdown
Contributor Author

@smorimoto

@smorimoto

Copy link
Copy Markdown
Member

Could you please rebase this?

@toastal

toastal commented May 27, 2025

Copy link
Copy Markdown
Contributor Author

Local said I was up to date. I will force push again… it could have been a miss timing.

@toastal

toastal commented May 27, 2025

Copy link
Copy Markdown
Contributor Author
$ git checkout microdata 
Switched to branch 'microdata'
$ git pull --rebase upstream master
From https://github.com/ocsigen/tyxml
 * branch              master     -> FETCH_HEAD
Current branch microdata is up to date.
$ dune clean
$ dune test --stop-on-first-error
File "ppx/tyxml_ppx.ml", line 381, characters 18-65:
381 |   | Pexp_function (params, constraint_, (Pfunction_body content)) ->
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Error: This pattern matches values of type 'a * 'b * 'c
       but a pattern was expected which matches values of type case list

@smorimoto this is the same error the CI was giving

@toastal

toastal commented Jun 12, 2025

Copy link
Copy Markdown
Contributor Author

reping @Drup

@toastal toastal force-pushed the microdata branch 2 times, most recently from af04e28 to 4975c49 Compare June 21, 2025 16:45
@toastal

toastal commented Jun 21, 2025

Copy link
Copy Markdown
Contributor Author

@smorimoto @Drup I added 2 commits. 1) found a typo 2) trying to add in flow content which is what the spec says: https://html.spec.whatwg.org/multipage/dom.html#flow-content

That last bit, I’m very unsure of my design with constraints with the OCaml type system, but <meta itemprop> needs to be allowable & is common for use with microdata.

@toastal toastal force-pushed the microdata branch 3 times, most recently from 29f8958 to 049f309 Compare June 21, 2025 17:03
@toastal

toastal commented Jun 21, 2025

Copy link
Copy Markdown
Contributor Author

I would like to run the test suite but I can’t with

$ dune test
File "ppx/tyxml_ppx.ml", line 381, characters 18-65:
381 |   | Pexp_function (params, constraint_, (Pfunction_body content)) ->
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Error: This pattern matches values of type 'a * 'b * 'c
       but a pattern was expected which matches values of type case list

Tried switching to OCaml 5.2 as well to no avail.

@toastal

toastal commented Jun 21, 2025

Copy link
Copy Markdown
Contributor Author

Cherry-picking out typos & syntax errors to keep focus here

@toastal toastal force-pushed the microdata branch 4 times, most recently from b14d59f to 5edca56 Compare December 2, 2025 03:23
@toastal

toastal commented Dec 13, 2025

Copy link
Copy Markdown
Contributor Author

ping @Drup

@toastal

toastal commented Dec 13, 2025

Copy link
Copy Markdown
Contributor Author

Rebasing…

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.

2 participants