-
Notifications
You must be signed in to change notification settings - Fork 1
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 for Documentation and tests #10
Conversation
Nice! I think it is sufficient if we only test on 1.6 and nightly since 1.5 is no longer maintained and 1.0.5 LTS will soon (at the time of the 1.7 release in about 2-3 months) be replaced by 1.6 as LTS. |
When the checks have finished I guess everything works and it is ready to be merged (this time with squash/merge 😉) |
I would keep it as it is for now and with the release of 1.7 and the change of the LTS also change the CI. |
Looks good to me. I'll merge and look at code coverage in a separate PR. |
This should add the needed CI for creating docs on push/pull requests to master as discussed in #5.