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 VS based build #234

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Remove VS based build #234

merged 1 commit into from
Nov 18, 2024

Conversation

Alan-Jowett
Copy link
Member

This pull request primarily involves the removal of multiple build configuration files and their associated filters. These changes simplify the project structure by eliminating redundant or outdated configurations.

Key changes include:

Removal of build configurations and steps:

  • build-vs job and its associated steps have been removed from .github/workflows/build.yml. This includes matrix configurations, environment variables, and various build and test steps.

Removal of project files:

  • cxplat_test.vcxproj and cxplat_test.vcxproj.filters files have been deleted from the cxplat/cxplat_test directory. These files contained configurations for building and filtering source files for the cxplat_test project. [1] [2]

  • cxplat_winkernel.vcxproj and cxplat_winkernel.vcxproj.filters files have been deleted from the cxplat/src/cxplat_winkernel directory. These files contained configurations for building and filtering source files for the cxplat_winkernel project. [1] [2]

  • cxplat_winuser.vcxproj file has been deleted from the cxplat/src/cxplat_winuser directory. This file contained configurations for building the cxplat_winuser project.

Signed-off-by: Alan Jowett <[email protected]>
@Alan-Jowett Alan-Jowett merged commit 0305975 into microsoft:main Nov 18, 2024
5 checks passed
Alan-Jowett added a commit that referenced this pull request Nov 19, 2024
Alan-Jowett added a commit that referenced this pull request Nov 19, 2024
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