[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This change introduces some build system fixes as well as adding VMA to our dependency list. This particular PR will not actually invoke anything, this is to just get the ball rolling.
Is there an open issue that this resolves? If so, please link it here.
No, but is part of a graphics API overhaul I am doing.
What is the current behavior? (You can also link to an open issue here)
VMA does not exist in the NovelRT project.
What is the new behavior (if this is a feature change)?
VMA exists in the NovelRT project.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
The build system no longer builds the dependencies as a separate CMake process.
FiniteReality
commented
1 year ago
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) This change introduces some build system fixes as well as adding VMA to our dependency list. This particular PR will not actually invoke anything, this is to just get the ball rolling.
Is there an open issue that this resolves? If so, please link it here. No, but is part of a graphics API overhaul I am doing.
What is the current behavior? (You can also link to an open issue here) VMA does not exist in the NovelRT project.
What is the new behavior (if this is a feature change)? VMA exists in the NovelRT project.
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) The build system no longer builds the dependencies as a separate CMake process.
Other information: N/A