-
Notifications
You must be signed in to change notification settings - Fork 46
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
CI Failing #104
Comments
Hi @asalmgren if you're referring to the badge in the README, this is due to an incorrect link. I've fixed that in #111. |
Raf -- thanks for fixing that! Looking at the CI testing now ... how long
does it usually take to complete the CI for a PR?
…On Wed, Oct 27, 2021 at 8:34 AM Rafael M Mudafort ***@***.***> wrote:
Hi @asalmgren <https://github.com/asalmgren> if you're referring to the
badge in the README, this is due to an incorrect link. I've fixed that in
#111 <#111>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#104 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACRE6YSPADLIXPPBRGEPHYTUJALYXANCNFSM5GZCL7OQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Ann Almgren
Senior Scientist; CCSE Group Lead
Pronouns: she/her/hers
|
This has been fixed |
@asalmgren The CI walltime is something that we should discuss. It's an issue @rafmudaf identified, with each problem recompiling the same source code instead of linking to a common erf library. |
Yes, I was noticing that, and I think we can consolidate some of the tests
for now at least. But this will be a bigger and bigger issue going forward
…On Thu, Oct 28, 2021 at 12:17 PM ewquon ***@***.***> wrote:
@asalmgren <https://github.com/asalmgren> The CI walltime is something
that we should discuss. It's an issue @rafmudaf
<https://github.com/rafmudaf> identified, with each problem recompiling
the same source code instead of linking to a common erf library.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#104 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACRE6YRXRGXU3DIOK56VYGTUJGOVVANCNFSM5GZCL7OQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Ann Almgren
Senior Scientist; CCSE Group Lead
Pronouns: she/her/hers
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Why do we have CI failing?
The text was updated successfully, but these errors were encountered: