-
Notifications
You must be signed in to change notification settings - Fork 68
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
Ensure spec PR's diffs are generated correctly #455
Comments
I traced where the spurious diff changes are coming from. Bikeshed generates IDs for term references for the definition panels using "circled digits" ( Unfortunately, I don't think this can be changed. Setting Use Dfn Panels to false still makes Bikeshed generate the IDs, and the actual diff engine doesn't have any relevant configuration. I'm not sure anything can be done about it, short of writing a new postprocessor and registering it with pr-preview. |
CC @domenic given your experience with writing specs, did you stumble over that and can recommend a fix? |
I don't really use the diffs, sorry. |
You could try patching https://github.com/w3c/htmldiff-ui but I'd recommend just skipping past the noise. (In particular, to do a thorough review you need to look at the source changes so you can use those as a guide as to what constitutes noise.) |
See #450 (comment).
The text was updated successfully, but these errors were encountered: