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

[Feature Request] Setup CI #38

Closed
HLWeil opened this issue Sep 11, 2021 · 1 comment
Closed

[Feature Request] Setup CI #38

HLWeil opened this issue Sep 11, 2021 · 1 comment

Comments

@HLWeil
Copy link
Member

HLWeil commented Sep 11, 2021

Successful building of the library is an absolute minimum criterion for all changes pushed to this repository. To ensure this CI should be added.

Building and testing similar to the one performed in BioFSharp

@HLWeil HLWeil mentioned this issue Sep 12, 2021
@HLWeil
Copy link
Member Author

HLWeil commented Oct 22, 2021

Merged #39 despite it failing atm. The windows build runs fine and will be a better indicator of build success compared to the previous nothing. Building locally in Ubuntu 20.04 LTS worked fine and the linux build steps also run fine in the CI, only failing at the runTests step because of the test project not being able to load the built ISADotNet library correctly:

Error: /home/runner/work/ISADotNet/ISADotNet/tests/ISADotNet.Tests/ISADotnet/JsonExtensionsTests.fs(15,7): error FS0039: The type 'AnyOf' is not defined. Maybe you want one of the following:�   AnyOfType [/home/runner/work/ISADotNet/ISADotNet/tests/ISADotNet.Tests/ISADotNet.Tests.fsproj]

HLWeil added a commit that referenced this issue Apr 22, 2022
#38 this should definitely be looked into again at a later point
@HLWeil HLWeil closed this as completed Nov 29, 2022
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

No branches or pull requests

1 participant