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

Move to Github Actions, added some simple tests #8

Merged
merged 5 commits into from
Apr 16, 2021
Merged

Conversation

jagot
Copy link
Collaborator

@jagot jagot commented Apr 12, 2021

I moved the CI pipeline to Github Actions and added the usual suspects so we can keep compats up to date.

I also added some simple tests based on the example in the documentation, perhaps you could check if they make sense.

For documentation to deploy, you need to generate a new secret DOCUMENTER_KEY according to:
https://juliadocs.github.io/Documenter.jl/stable/man/hosting/#GitHub-Actions

@jagot
Copy link
Collaborator Author

jagot commented Apr 12, 2021

It seems the Github Actions do not run on this PR, perhaps you need to enable them somehow? In any case, here is the status of the pipeline, running on my fork: https://github.com/jagot/JacobiDavidson.jl/actions/runs/740403717

@haampie
Copy link
Owner

haampie commented Apr 16, 2021

Thank you, this is a good idea!

@haampie
Copy link
Owner

haampie commented Apr 16, 2021

Let me merge to see if that fixes it

@haampie haampie merged commit 7321c17 into haampie:master Apr 16, 2021
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.

2 participants