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

Use respec includes for copyright statements #2550

Merged
merged 13 commits into from
Mar 30, 2023

Conversation

mattgarrish
Copy link
Member

@mattgarrish mattgarrish commented Mar 29, 2023

Uses data-oninclude to run the processing step on the included copyrights.

I've also stripped fixIncludes from all the other imports. It looks like I copied the import markup for acknowledgements from dpub-aria, as that's where I finally found the function defined, and then it got copied around wherever we've needed another import. It's correcting references to the ARIA core spec there, but obviously does nothing at all here.


Preview | Diff

@mattgarrish
Copy link
Member Author

I think we might have to abandon this idea, unless you want to report it to get it fixed @iherman?

For whatever reason, the conversion process run by the github action makes a mess of the files when the copyright is included using the respec functions. It doesn't appear to be related to the modifyCopyright function call, either, as I get the same errors using a static import.

I'm guessing from the errors that the paragraph is being moved into the header, but why is beyond me as it must be happening in a step I can't reproduce. If I serve the files locally and export, there's no issue.

@mattgarrish
Copy link
Member Author

mattgarrish commented Mar 30, 2023

Sigh... well, at least I realized what a doofus I was before completely drifting off to sleep.

I forgot to commit the shared copyright file. I have no idea why that resulted in validation errors about duplicate meta charset tags and a title tag in the body, but whatever. All good now. Day thoroughly wasted chasing phantom bugs...

If you go ahead and merge this, it might be good to pick "squash and merge" from the options so all my test commits don't get combined into the spec history.

@iherman iherman merged commit 19953c9 into copyright-change-via-respec Mar 30, 2023
@iherman iherman deleted the use-respec-includes branch March 30, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants