-
Notifications
You must be signed in to change notification settings - Fork 193
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
Try to fix Broken C# bindings workflow due to Alfalfa addition #5291
Conversation
…_OpenStudio_LeftShift__SWIG_1'"
@jmarrec I have the fix for the build failing when building the test. Do you want me to push it here? |
@TShapinsky In the interest of efficiency, please always assume that is fine if I'm not on slack and it's nighttime in France and it's a quick fix (if you touch 1000s of lines, a PR pointing to the PR like you did is better). I can always revert back or adjust if not. In this specific case, a code suggestion (since I did touch these lines) would have worked too. |
Fix namesapce for types in GTest
CI Results for c666655:
|
Pull request overview
Pull Request Author
src/model/test
)src/energyplus/Test
)src/osversion/VersionTranslator.cpp
)Labels:
IDDChange
APIChange
Pull Request - Ready for CI
so that CI builds your PRReview Checklist
This will not be exhaustively relevant to every PR.