Skip to content

fix: Update CEF download link in CI workflow#1

Merged
DealsBeam merged 2 commits intomainfrom
add-build-script
Aug 30, 2025
Merged

fix: Update CEF download link in CI workflow#1
DealsBeam merged 2 commits intomainfrom
add-build-script

Conversation

@google-labs-jules
Copy link
Copy Markdown

This pull request corrects an error in the recently added CI pipeline. The download URL for the Chromium Embedded Framework (CEF) dependency was pointing to an incorrect version (139).

This has been updated to the correct version 136, which is the version specified in the BUILDING.md file. The cache key has also been updated to match.

This ensures the CI pipeline is building against the correct and intended version of the project's dependencies.

This commit introduces a new Windows batch script, `build.bat`, to fully automate the build process described in `BUILDING.md`.

The script handles:
- Cloning and bootstrapping vcpkg
- Installing required vcpkg packages
- Building the Chromium Embedded Framework (CEF) from a user-provided directory
- Installing NPM dependencies for the UI
- Creating and installing a test certificate
- Building the main Visual Studio solution

This script significantly simplifies the setup for new and existing developers, reducing the chance of manual errors.

The `BUILDING.md` file has also been updated to document the usage of this new script.
This commit corrects the download URL for the Chromium Embedded Framework (CEF) dependency in the GitHub Actions CI workflow.

The previous URL pointed to version 139, which was incorrect. This has been updated to point to the correct version 136, as specified in the project's build documentation.

The cache key for the CEF dependency has also been updated from `v139` to `v136` to reflect this change.
@DealsBeam DealsBeam marked this pull request as ready for review August 30, 2025 21:29
@DealsBeam DealsBeam merged commit 8995a05 into main Aug 30, 2025
1 check failed
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