Skip to content
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

TST: Add testing to github action #173

Closed
wants to merge 4 commits into from
Closed

Conversation

ku-ya
Copy link
Contributor

@ku-ya ku-ya commented Nov 1, 2020

Add cron job with github actions to schedule test runs monthly/fortnightly.

Contributes to #163

@ku-ya ku-ya marked this pull request as ready for review November 1, 2020 06:13
@GenevieveBuckley
Copy link
Collaborator

I'm going to take a look at this later, thank you @ku-ya

@GenevieveBuckley
Copy link
Collaborator

Update: I've been trying to see if I can make this run every 15 minutes, but unfortunately that doesn't seem to be working on my branch? (This is the first time I have tried to schedule github actions, I might be making some mistakes)

After we can confirm it works, we can make the interval once per month and merge.

Base automatically changed from master to main February 2, 2021 01:18
@jakirkham
Copy link
Member

Thanks for the help @ku-ya! 🙏

We moved to GitHub Actions in PR ( #188 ). That said, if you spot anything we should add, please feel free to either update this PR to apply changes on latest main or open a new PR, whichever is easier.

Thanks again! 😄

@GPUtester
Copy link
Collaborator

Can one of the admins verify this patch?

Admins can comment ok to test to allow this one PR to run or add to allowlist to allow all future PRs from the same author to run.

@GenevieveBuckley
Copy link
Collaborator

I remember this from the sprint. There have been a few changes since then to the way we handle CI, so it's probably best to close this out.

Thanks for your code sprint contributions @ku-ya, it was great having you there 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants