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
switch to GitHub Actions #34
Conversation
|
This looks amazing! I'd love to switch to GA. |
It currently uses nightly, but can be switched. See this line |
Got it. Could we switch to Another thing I wanted to ask, if you could run all the examples in the CI. IMHO it will be a nice addition to tests, to make sure examples do no produce runtime errors. |
sure, but I'll have to change my live plot example to only run for a
limited time. Maybe like ~5s?
…On Mon, Aug 23, 2021 at 12:32 AM Alexey Suslov ***@***.***> wrote:
Got it. Could we switch to stable please? since we are not using any
nightly features.
Another thing I wanted to ask, if you could run all the examples in the
CI. IMHO it will be a nice addition to tests, to make sure examples do no
produce runtime errors.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#34 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC5YNWVDSOCCCJYKB32KWV3T6H2RDANCNFSM5CSJFKDQ>
.
|
5s is fine 👍 |
This is working and good to merge! |
Thanks @TDHolmes! |
No worries if you want to stick with Travis for CI, but thought I'd throw it up to see what you thought. Since it's coming from a fork you won't see the CI running, but see this PR on my fork for what it'd look like for future PRs if you merged.