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 azure canvas backend #25833

Closed
jdm opened this issue Feb 24, 2020 · 6 comments
Closed

Remove azure canvas backend #25833

jdm opened this issue Feb 24, 2020 · 6 comments

Comments

@jdm
Copy link
Member

@jdm jdm commented Feb 24, 2020

Now that the raqote backend has reached feature parity and been enabled by default, we should remove the unused azure backend entirely.

@highfive
Copy link

@highfive highfive commented Feb 24, 2020

Hi! If you have any questions regarding this issue, feel free to make a comment here, or ask it in the #servo channel in IRC.

If you intend to work on this issue, then add @highfive: assign me to your comment, and I'll assign this to you. 😄

@dralley
Copy link
Contributor

@dralley dralley commented Feb 25, 2020

@highfive: assign me

@highfive highfive added the C-assigned label Feb 25, 2020
@highfive
Copy link

@highfive highfive commented Feb 25, 2020

Hey @dralley! Thanks for your interest in working on this issue. It's now assigned to you!

@dralley
Copy link
Contributor

@dralley dralley commented Feb 25, 2020

Currently you can choose to use either the Azure backend or the Raquote backend, but at least one of them and not both. I assume that as part of this issue, the feature flag that allows the Raquote backend to be disabled should be removed?

@jdm
Copy link
Member Author

@jdm jdm commented Feb 25, 2020

Correct!

@dralley dralley mentioned this issue Feb 25, 2020
2 of 5 tasks complete
@dralley
Copy link
Contributor

@dralley dralley commented Feb 25, 2020

PR posted

bors-servo added a commit that referenced this issue Feb 25, 2020
Remove azure canvas backend

closes #25833

Issue description: Now that the raqote backend has reached feature parity and been enabled by default, we should remove the unused azure backend entirely.

<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [ ] `./mach build -d` does not report any errors
- [ ] `./mach test-tidy` does not report any errors
- [x] These changes fix #25833

<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not require tests because PR only removes code, doesn't add new code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.