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 dependency on webrender_api #92

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Remove dependency on webrender_api #92

merged 1 commit into from
Nov 25, 2024

Conversation

nicoburns
Copy link
Collaborator

@nicoburns nicoburns commented Nov 23, 2024

I have submitted this upstream: https://bugzilla.mozilla.org/show_bug.cgi?id=1933010

We can actually land the Servo PR without landing this first. But we might want to land anyway this to ensure that Servo doesn't regress before the next stylo sync.

Servo PR: servo/servo#34353

@nicoburns nicoburns added this pull request to the merge queue Nov 25, 2024
@nicoburns
Copy link
Collaborator Author

This has been accepted (but not yet landed) upstream. But I'm going to land it here to so we can get the Servo PR merged (and make sure it doesn't regress before we next sync Stylo).

Merged via the queue into main with commit 4eb9da3 Nov 25, 2024
3 checks passed
@Loirooriol Loirooriol deleted the remove_webrender_api branch November 25, 2024 21:46
@nicoburns nicoburns restored the remove_webrender_api branch November 25, 2024 22:40
@Loirooriol
Copy link
Contributor

When you open a PR, can you use a branch from your fork? Or at least delete it afterwards. It's weird if the repo accumulates branches from past PRs.

@Loirooriol Loirooriol deleted the remove_webrender_api branch December 5, 2024 21:38
@nicoburns
Copy link
Collaborator Author

Sure! We could also turn on the option to automatically delete branches of merged PRs.

@Loirooriol
Copy link
Contributor

Loirooriol commented Dec 5, 2024

Ah, makes sense. Done.

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