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

Adds a github action workflow. #241

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

hakonhagland
Copy link
Contributor

This PR builds on #240, which should be merged first.

Adds a GitHub action workflow as an alternative to the travis build, see issue #235. The current workflow tests against perl version 5.34, 5.32, 5.30, 5.28, and 5.26, and GSL versions 2.7, 2.6, 2.5, 2.4, and 2.3 which give 5*5=25 test cases.

This also solves #235.

@leto If you merge this, we should be able to see build results. If the tests passes the next step would be to upload a new version of the module to CPAN.

First step is to make perl Build.PL run. After this change I am able
to run "perl Build.PL" and then "./Build" without failures.
According to new information only the unsigned versions of
gsl_spmatrix's norm1 function are missing.
Add the new functions introduced in 2.7 to Ver2Func.pm
Also fixes issue leto#239
Adds a GitHub action workflow to test the generated dist against various
combinations of perl and GSL.
@hakonhagland hakonhagland changed the title Adds github a action workflow. Adds a github action workflow. Jun 28, 2021
@hakonhagland
Copy link
Contributor Author

Anybody knows what happened to @leto? According to his GitHub page there has not been any activity the last 8 months which seems not right.

@hakonhagland
Copy link
Contributor Author

hakonhagland commented Jul 21, 2021

It seems all tests have passed for this PR in my fork. I am continuing the development in a new repository where I have write access. The plan is then to also merge PR #229 and then maybe do some testing on macOS and Windows before uploading a new release to CPAN. Please let me know if this sounds ok.

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.

1 participant