Skip to content

script: Move reporting interfaces#43270

Merged
TimvdLippe merged 1 commit intoservo:mainfrom
TimvdLippe:move-reporting
Mar 15, 2026
Merged

script: Move reporting interfaces#43270
TimvdLippe merged 1 commit intoservo:mainfrom
TimvdLippe:move-reporting

Conversation

@TimvdLippe
Copy link
Copy Markdown
Contributor

Also moves some relevant methods from GlobalScope that were only used within ReportingObserver.

Part of #38901

Testing: It compiles

@TimvdLippe TimvdLippe requested a review from gterzian as a code owner March 14, 2026 18:27
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 14, 2026
@servo-highfive servo-highfive added S-needs-rebase There are merge conflict errors. and removed S-awaiting-review There is new code that needs to be reviewed. labels Mar 15, 2026
Also moves some relevant methods from GlobalScope that
were only used within ReportingObserver.

Part of servo#38901

Signed-off-by: Tim van der Lippe <[email protected]>
@servo-highfive servo-highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-needs-rebase There are merge conflict errors. labels Mar 15, 2026
@TimvdLippe TimvdLippe enabled auto-merge March 15, 2026 08:52
@TimvdLippe TimvdLippe added this pull request to the merge queue Mar 15, 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 15, 2026
Merged via the queue into servo:main with commit f8b06d7 Mar 15, 2026
33 checks passed
@TimvdLippe TimvdLippe deleted the move-reporting branch March 15, 2026 09:35
@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 15, 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.

3 participants