Skip to content

chore: Remove Wef code as it has been extracted to separate repository#1302

Merged
huacnlee merged 5 commits intomainfrom
copilot/fix-d5e2c3d9-f280-4f6d-8343-63423394a6fe
Sep 29, 2025
Merged

chore: Remove Wef code as it has been extracted to separate repository#1302
huacnlee merged 5 commits intomainfrom
copilot/fix-d5e2c3d9-f280-4f6d-8343-63423394a6fe

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 29, 2025

All WEF (Web Embedding Framework) related code has been successfully removed from the repository as it has been extracted to https://github.com/longbridge/wef

Copilot AI and others added 2 commits September 29, 2025 06:43
Co-authored-by: huacnlee <5518+huacnlee@users.noreply.github.com>
Co-authored-by: huacnlee <5518+huacnlee@users.noreply.github.com>
Copilot AI changed the title [WIP] Make a PR to remove wef relative code. Because I have extract it to @longbridge/wef/pull/1 Remove WEF (Web Embedding Framework) code as it has been extracted to separate repository Sep 29, 2025
Copilot AI requested a review from huacnlee September 29, 2025 06:50
@huacnlee huacnlee changed the title Remove WEF (Web Embedding Framework) code as it has been extracted to separate repository chore: Remove Wef code as it has been extracted to separate repository Sep 29, 2025

```toml
gpui-component = { git = "https://github.com/longbridge/gpui-component.git", features = ["webview"] }
```
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot We need keep this part.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're absolutely right! I've restored the WebView documentation section. The wry-based WebView functionality in the UI crate is separate from WEF and should remain available. Restored in commit 7b5efd1.

@huacnlee
Copy link
Copy Markdown
Member

@copilot And the ci.yml should to change to remove the wef test.

Co-authored-by: huacnlee <5518+huacnlee@users.noreply.github.com>
Co-authored-by: huacnlee <5518+huacnlee@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Sep 29, 2025

@copilot And the ci.yml should to change to remove the wef test.

Updated the CI configuration to remove all WEF-related steps including CEF_VERSION environment variable, CEF caching, WEF initialization, and cargo-wef commands. The CI now runs clean tests without WEF dependencies. Fixed in commit 5a3bbdd.

@huacnlee huacnlee marked this pull request as ready for review September 29, 2025 07:31
@huacnlee huacnlee merged commit ed05c3e into main Sep 29, 2025
2 checks passed
@huacnlee huacnlee deleted the copilot/fix-d5e2c3d9-f280-4f6d-8343-63423394a6fe branch September 29, 2025 07:32
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