-
Notifications
You must be signed in to change notification settings - Fork 447
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
Azure Linux Builds #1752
Azure Linux Builds #1752
Conversation
ee839d8
to
16bf6a4
Compare
conda install qcelemental ^ | ||
qcengine ^ | ||
pybind11 ^ | ||
-c conda-forge |
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.
why c-f not psi4/label/dev?
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.
I pulled this from Travis.
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.
figure out why it's from c-f. this makes sure psi is using latest, even if I'm behind on psi4-dev packages.
a99004f
to
5401c94
Compare
0388a23
to
2dafcca
Compare
This pull request introduces 1 alert when merging 2dafcca into dfb86f5 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging f40c04b into dfb86f5 - view on LGTM.com new alerts:
|
6cdc6bc
to
b569f69
Compare
This pull request introduces 1 alert when merging 59bf004 into dfb86f5 - view on LGTM.com new alerts:
|
dbb3d2b
to
d6a6baa
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.
Awesome! Thanks a lot! 🎉 🎊
@psi4/admins Ready to review! |
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.
this is great! maybe we can run codecov on PRs and not leave full test testing only to conda-build time.
I'll close #1748, but is the plan to use Ninja also on Linux/macOS? |
It didn't seem to help much on Travis or am I wrong there? Very on board with ninja overall, just balancing the complexity. |
That's correct, not much help on Travis. I was under the impression it was going to be a requirement when moving to libint2. But I can have a go at it later, once this is merged. |
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.
Nice!
Someone should let Travis know they're not required anymore 😺 |
Travis and GitHub has been informed! |
Description
The plan is to switch away from TravisCI Linux builds to Azure for unlimited build minutes, the ability to host more complex pipelines, and to prevent splitting between multiple services.
Status