Skip to content

script: Move xml interfaces into xml/#43604

Closed
jayant911 wants to merge 2 commits into
servo:mainfrom
jayant911:move-xml-interfaces
Closed

script: Move xml interfaces into xml/#43604
jayant911 wants to merge 2 commits into
servo:mainfrom
jayant911:move-xml-interfaces

Conversation

@jayant911

Copy link
Copy Markdown
Contributor

Moves xml interfaces into script/dom/xml/ module from script/dom/.

Testing: Just a refactor shouldn't need any testing.
Fixes: Part of #38901

Signed-off-by: Jayanta Pradhan <pradhanjayanta91@gmail.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 24, 2026
@jayant911

jayant911 commented Mar 24, 2026

Copy link
Copy Markdown
Contributor Author

@mukilan, @eerii! please guide me to resolve this.

@mukilan

mukilan commented Mar 24, 2026

Copy link
Copy Markdown
Member

@jayant911 I'll take a look at this soon.

Signed-off-by: Jayanta Pradhan <pradhanjayanta91@gmail.com>
Comment thread components/script/dom/xml/mod.rs
@servo-highfive servo-highfive added the S-needs-rebase There are merge conflict errors. label Mar 25, 2026
@jayant911 jayant911 closed this Mar 25, 2026
@mukilan

mukilan commented Mar 25, 2026

Copy link
Copy Markdown
Member

@jayant911 Apologies for the confusion. You can still mention this PR in your Outreachy contributions and add the context.

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. S-needs-rebase There are merge conflict errors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants