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

Added support for KHR_materials_pbrSpecularGlossiness. #31

Merged

Conversation

DragonJoker
Copy link
Contributor

No description provided.

@DragonJoker DragonJoker force-pushed the add_KHR_materials_pbrSpecularGlossiness branch from 183650f to 22952b6 Compare September 7, 2023 06:11
@DragonJoker DragonJoker force-pushed the add_KHR_materials_pbrSpecularGlossiness branch from 22952b6 to 8e53e7e Compare September 7, 2023 06:12
@spnda
Copy link
Owner

spnda commented Sep 7, 2023

I guess it would also make sense to add a Actions runner that would also run the tests with the define enabled. Right now, the tests never get built through CI for the extension.
If you're unsure how to do that, I can also take care of it after the PR is merged.

include/fastgltf/types.hpp Outdated Show resolved Hide resolved
@spnda spnda merged commit 46f28f1 into spnda:main Sep 7, 2023
11 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.

None yet

2 participants