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.

Sign up for GitHub

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

Jump to bottom

Add devel to CI #99

Merged
merged 17 commits into from
Jun 30, 2020
Merged

Add devel to CI #99

merged 17 commits into from
Jun 30, 2020

Conversation

0atman
Copy link
Collaborator

@0atman 0atman commented Jun 30, 2020 •
edited
Loading

As per #89

@0atman 0atman linked an issue Jun 30, 2020 that may be closed by this pull request
nimble test fails #89
Closed
@0atman 0atman added the enhancement New feature or request label Jun 30, 2020
@0atman 0atman self-assigned this Jun 30, 2020
@0atman 0atman mentioned this pull request Jun 30, 2020
Closed
@0atman
Copy link
Collaborator Author

0atman commented Jun 30, 2020

using choosenim for devel requires recompiling each test - what was a 20s test is now 5 minutes 😢

@0atman
Copy link
Collaborator Author

0atman commented Jun 30, 2020

@timotheecour are you aware of a workaround for compiling our own version of nim? the devel tag on the nim docker image hasn't been built for years

@0atman
Copy link
Collaborator Author

0atman commented Jun 30, 2020

Phew, that was not fun.

@0atman 0atman merged commit 8cbaa12 into master Jun 30, 2020
@Tangdongle Tangdongle mentioned this pull request Sep 7, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timotheecour timotheecour timotheecour left review comments

Assignees

@0atman 0atman

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

nimble test fails
2 participants
@0atman @timotheecour