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

Convert Travis to GHA #21

Merged
merged 7 commits into from
Mar 10, 2021
Merged

Convert Travis to GHA #21

merged 7 commits into from
Mar 10, 2021

Conversation

christopher-dG
Copy link
Member

@christopher-dG christopher-dG commented Mar 5, 2021

Closes #18

@hannahilea
Copy link
Contributor

...i'll fix the test failure on monday. :)

matrix:
version:
- '1.3'
- '1'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...so the reason this is failing is that we previously supported 1.3 and 1.4, and now we're supporting 1.3 and 1 (...aka 1.5). There's a dependence on a conveniently chosen seeded rng in the tests that (due to intentional/known) differences between rngs across Julia versions causes a failure in 1.5.

The correct ultimate fix is to remove that test dependence on the rng, but that fix is orthogonal to the changes introduced by this PR, so I think a good course of action here is to not change the supported Julia versions here (i.e., revert to 1.3 and 1.4) and then file a separate issue for bumping the supported versions to 1.3 and 1 (which would necessarily require fixing that test).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...lol, that issue is in fact already filed (as #18).

.github/workflows/CI.yml Outdated Show resolved Hide resolved
@christopher-dG
Copy link
Member Author

Added a scheduled run every 2 weeks since @hannahilea mentioned that might be a nice thing to have.

@hannahilea
Copy link
Contributor

Closes #18.

@hannahilea hannahilea merged commit f872e6f into master Mar 10, 2021
@hannahilea hannahilea deleted the cdg/gha branch March 10, 2021 23:23
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.

Do not depend on finding the perfect rng seed for the learn test
2 participants