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

deps.windows: Add WIL headers #242

Merged
merged 1 commit into from
May 2, 2024
Merged

Conversation

thyintel
Copy link
Contributor

@thyintel thyintel commented Apr 5, 2024

Description

Add wil v1.0.240122.1 headers for win-mf.
Linked with PR: obsproject/obs-studio#10471

Motivation and Context

Adds wil headers to be used with win-mf project in obs-studio.

How Has This Been Tested?

Testing is described in the linked obs-studio PR.

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • [ x] My code has been run through clang-format.
  • [ x] I have read the contributing document.
  • [ x] My code is not on the master branch.
  • [ x] The code has been tested.
  • [ x] All commit messages are properly formatted and commits squashed where appropriate.
  • [ x] I have included updates to all appropriate documentation.

Copy link
Member

@RytoEX RytoEX left a comment

Choose a reason for hiding this comment

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

This seems okay at a glance, and is required for obsproject/obs-studio#10471 to work.

I'd probably personally have lowercased "headers" in the commit message, but that's not a blocker.

@thyintel
Copy link
Contributor Author

This seems okay at a glance, and is required for obsproject/obs-studio#10471 to work.

I'd probably personally have lowercased "headers" in the commit message, but that's not a blocker.

Changed it to lowercase!

@thyintel thyintel changed the title deps.windows: Add WIL Headers deps.windows: Add WIL headers Apr 22, 2024
@RytoEX RytoEX merged commit d77d7f4 into obsproject:master May 2, 2024
21 checks passed
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