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

Remove more definitions of MutationEvent and friends #382

Merged
merged 8 commits into from
Sep 7, 2024

Conversation

mfreed7
Copy link
Contributor

@mfreed7 mfreed7 commented Sep 6, 2024

See #381 for the first part of this cleanup.

I really don't understand how this repo is structured, it seems like tons of generated files are checked in, and aren't actually built by the build.py. I ended up just manually deleting mutation events from all the places, hopefully correctly.

The following tasks have been completed:

  • Confirmed there are no ReSpec/BikeShed errors or warnings.
  • Modified Web platform tests (link to pull request)

web-platform-tests/wpt#47727

Implementation commitment:

@garykac
Copy link
Member

garykac commented Sep 6, 2024

Argh! I thought this PR had already been merged before I deleted the obsolete files.

Also, note that the README file has a "To make edits to the spec" section that identifies the files that need to be updated.

@mfreed7
Copy link
Contributor Author

mfreed7 commented Sep 7, 2024

Argh! I thought this PR had already been merged before I deleted the obsolete files.

No problem. I believe I just rebased on top of all of the deletions, but could you take a look?

Also, note that the README file has a "To make edits to the spec" section that identifies the files that need to be updated.

Right, I saw that, but it sounds from the wording like there shouldn't be target files (e.g. TR.html) included in the committed tree. And locally running build.py didn't seem to update any of them. So I was confused.

@garykac garykac merged commit 02207f8 into w3c:main Sep 7, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Sep 7, 2024
SHA: 02207f8
Reason: push, by garykac

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants