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

Update CI files to include requested changes #394

Merged
merged 8 commits into from
Mar 16, 2022
Merged

Update CI files to include requested changes #394

merged 8 commits into from
Mar 16, 2022

Conversation

ellie-idb
Copy link
Member

See libmir/mir-ion#16 for more detail.

@9il
Copy link
Member

9il commented Mar 12, 2022

Please get rid of appveyor

@9il
Copy link
Member

9il commented Mar 12, 2022

Some CI jobs fails becaues of runners goes to out of memory, especially windows.
Possible solution is:

  • Either split ndslice, bignum and the rest of ndslice tests.
  • Or use -lowmem flag. I am not sure if it works for DMD.

@ellie-idb
Copy link
Member Author

Please get rid of appveyor

I'm personally not able to get rid of AppVeyor -- it's under your account, so you'll have to do that

@ellie-idb ellie-idb force-pushed the github-actions branch 5 times, most recently from c6b7423 to 7c9d840 Compare March 14, 2022 14:05
@jmh530
Copy link
Contributor

jmh530 commented Mar 14, 2022

Would there be any benefit to moving unit tests that are not documentation tests to a separate folder by themselves?

@9il 9il merged commit 84b3c24 into master Mar 16, 2022
@9il
Copy link
Member

9il commented Mar 16, 2022

Would there be any benefit to moving unit tests that are not documentation tests to a separate folder by themselves?

I don't think so. It is usefull to have then right in the file.

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.

3 participants