Skip to content

script: Start preparation to use safer JSContext#40465

Merged
sagudev merged 2 commits intoservo:mainfrom
sagudev:safe-mozjs-cx
Nov 9, 2025
Merged

script: Start preparation to use safer JSContext#40465
sagudev merged 2 commits intoservo:mainfrom
sagudev:safe-mozjs-cx

Conversation

@sagudev
Copy link
Copy Markdown
Member

@sagudev sagudev commented Nov 6, 2025

This PR is companion to servo/mozjs#638, currently it only make stuff compile. In follwups we will start passing/using safer JSContext down everywhere.

Testing: Not needed as it's just refactorings.
try run: https://github.com/sagudev/servo/actions/runs/19196312972/job/54879090390

@sagudev sagudev force-pushed the safe-mozjs-cx branch 2 times, most recently from 1b63c76 to 437ef45 Compare November 7, 2025 06:46
@sagudev sagudev marked this pull request as ready for review November 7, 2025 07:14
@sagudev sagudev requested a review from gterzian as a code owner November 7, 2025 07:14
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 7, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 7, 2025
@sagudev sagudev added this pull request to the merge queue Nov 7, 2025
@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 Nov 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 7, 2025
@servo-highfive servo-highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Nov 7, 2025
@servo-highfive servo-highfive added S-awaiting-review There is new code that needs to be reviewed. S-needs-rebase There are merge conflict errors. and removed S-tests-failed The changes caused existing tests to fail. labels Nov 8, 2025
Signed-off-by: sagudev <[email protected]>
@servo-highfive servo-highfive removed the S-needs-rebase There are merge conflict errors. label Nov 9, 2025
@sagudev sagudev enabled auto-merge November 9, 2025 04:59
f
Signed-off-by: sagudev <[email protected]>
@sagudev sagudev added this pull request to the merge queue Nov 9, 2025
@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 Nov 9, 2025
Merged via the queue into servo:main with commit b73b908 Nov 9, 2025
32 checks passed
@sagudev sagudev deleted the safe-mozjs-cx branch November 9, 2025 06:03
@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 Nov 9, 2025
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.

3 participants