Skip to content

Extract gpui_mac out of gpui - #2874

Closed
osiewicz wants to merge 20 commits into
mainfrom
gpui_extract_platform
Closed

Extract gpui_mac out of gpui#2874
osiewicz wants to merge 20 commits into
mainfrom
gpui_extract_platform

Conversation

@osiewicz

@osiewicz osiewicz commented Aug 21, 2023

Copy link
Copy Markdown
Member

An old commit brushed off and brought up to date; this adds a new crate gpui_mac, which is pretty much gpui's platform::mac module extracted into a separate crate. Thanks to this it should be easier to tell what exactly needs to be done to add a support for new platform. As a bonus, since gpui_mac is not a dependency of gpui, we could start building core crates (like editor and such) without having to wait for crates like metal to be done building (once we somehow decouple gpui from media..).

Release Notes:

  • N/A

@osiewicz

osiewicz commented Aug 28, 2023

Copy link
Copy Markdown
Member Author

This PR slowed down a bit, as there are issues with gpui::test macro. Namely, for gpui crate itself, these tests have to reside outside of gpui itself (as we need a platform to run them) - hence I've started moving them to gpui_platform. However, these tests refer to private members/functions/structs, which makes the testing in a crate external to gpui trickier.
Yikes.

@osiewicz

osiewicz commented Oct 6, 2023

Copy link
Copy Markdown
Member Author

Closing this as we're gonna move off of gpui soon-ish.

@osiewicz osiewicz closed this Oct 6, 2023
osiewicz added a commit that referenced this pull request Feb 19, 2026
#2874 on steroids

Before you mark this PR as ready for review, make sure that you have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [ ] Done a self-review taking into account security and performance
aspects
- [ ] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- N/A

---------

Co-authored-by: Eric Holk <eric@zed.dev>
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
zed-industries#2874 on steroids

Before you mark this PR as ready for review, make sure that you have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [ ] Done a self-review taking into account security and performance
aspects
- [ ] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- N/A

---------

Co-authored-by: Eric Holk <eric@zed.dev>
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
zed-industries#2874 on steroids

Before you mark this PR as ready for review, make sure that you have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [ ] Done a self-review taking into account security and performance
aspects
- [ ] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- N/A

---------

Co-authored-by: Eric Holk <eric@zed.dev>
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.

1 participant