-
Notifications
You must be signed in to change notification settings - Fork 185
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
Make the "labs" cli implementation the default #5006
Conversation
The old implementation will be under the "classic" sub command. close #5005
Triggering CI on it. Assuming this comes clean, this can merge. |
3eb6887
to
b2f4719
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kbenne Fixed the tests for you. I think CI will be clean after that. I added a banner to the "classic" CLI, please review that
fmt::print(fmt::fg(fmt::color::orange), | ||
"┌{0:─^{2}}┐\n" | ||
"│{1: ^{2}}│\n" | ||
"└{0:─^{2}}┘", | ||
"", "The `classic` command is deprecated and will be removed in a future release", 80); | ||
fmt::print("\n"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kbenne I added a banner to the "classic" subcommand, maybe you disagree with that part though? This can be reverted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perf tests were failing due to this being omitted
@wenyikuang Seems like both the artifactory and the jenkins ci.openstudio.net itself certificates have just expired:
Certificate was set to expire today: |
confirmed, I cannot login either. Fixing. update: fixed. |
Respawing CI runs now that the certificate is properly updated on conan.openstudio.net |
CI Results for 52a8cf8:
|
The old implementation will be under the "classic" sub command.
close #5005
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.