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

CI: test with multiple Nim versions #106

Merged
merged 2 commits into from
Nov 18, 2021
Merged

CI: test with multiple Nim versions #106

merged 2 commits into from
Nov 18, 2021

Conversation

stefantalpalaru
Copy link
Contributor

No description provided.

@stefantalpalaru stefantalpalaru changed the title CI: test with multiple Nim versions [WIP] CI: test with multiple Nim versions Nov 17, 2021
@stefantalpalaru
Copy link
Contributor Author

The test runner is broken and doesn't exit with a non-zero value on test failure. Let's fix that first.

- plus Nim-1.6 fix
- and enable Windows
@stefantalpalaru stefantalpalaru changed the title [WIP] CI: test with multiple Nim versions CI: test with multiple Nim versions Nov 18, 2021
@stefantalpalaru stefantalpalaru merged commit 32e72c2 into master Nov 18, 2021
@stefantalpalaru stefantalpalaru deleted the ci_multi_nim branch November 18, 2021 19:04
arnetheduck pushed a commit that referenced this pull request Apr 7, 2022
* CI: test with multiple Nim versions

- plus Nim-1.6 fix
- and enable Windows

* size.nim: halve the number of log lines
arnetheduck pushed a commit that referenced this pull request Apr 8, 2022
* CI: test with multiple Nim versions

- plus Nim-1.6 fix
- and enable Windows

* size.nim: halve the number of log lines
arnetheduck added a commit that referenced this pull request Nov 17, 2023
* update readme

* Don't log non-printable characters

* bump mingw from v8.1.0 to v11.2.0

* ci: reenable windows matrix

* CI: test with multiple Nim versions (#106)

* CI: test with multiple Nim versions

- plus Nim-1.6 fix
- and enable Windows

* size.nim: halve the number of log lines

* enable styleCheck:usages (nbc branch) (#113)

* fix chronos v4 compatiblity (#138)

`yield` has never worked correctly in chronos - removing broken
function.

Also, the "official" way of checking the state of the future is via
`failed`

---------

Co-authored-by: Zahary Karadjov <zahary@gmail.com>
Co-authored-by: jangko <jangko128@gmail.com>
Co-authored-by: Ștefan Talpalaru <stefantalpalaru@yahoo.com>
Co-authored-by: Etan Kissling <etan@status.im>
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