Skip to content

script: Move document DOM interfaces to script/dom/document/#43602

Merged
yezhizhen merged 2 commits intoservo:mainfrom
elomscansio:move-document-interfaces
Mar 24, 2026
Merged

script: Move document DOM interfaces to script/dom/document/#43602
yezhizhen merged 2 commits intoservo:mainfrom
elomscansio:move-document-interfaces

Conversation

@elomscansio
Copy link
Copy Markdown
Contributor

Moves interfaces defined by the document spec to the script/dom/document/ module from script/dom/.

Testing: Just a refactor shouldn't need any testing
Fixes: Partially #38901

@elomscansio elomscansio requested a review from gterzian as a code owner March 24, 2026 09:55
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 24, 2026
@elomscansio elomscansio force-pushed the move-document-interfaces branch from 4308779 to 018b64b Compare March 24, 2026 10:19
@elomscansio
Copy link
Copy Markdown
Contributor Author

I followed steps on already merged PR #40262

@jdm @mrobinson @TimvdLippe

@elomscansio elomscansio force-pushed the move-document-interfaces branch from 018b64b to 823bee8 Compare March 24, 2026 10:28
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need to mention people for this kind of general change. They will be picked up automatically.

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 24, 2026
Signed-off-by: Emmanuel Paul Elom <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 24, 2026
@elomscansio elomscansio requested a review from TimvdLippe March 24, 2026 12:38
@yezhizhen yezhizhen added this pull request to the merge queue Mar 24, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 24, 2026
Merged via the queue into servo:main with commit 7e97e82 Mar 24, 2026
33 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants