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

Verify that rust version is recent enough to install rustlings. #131

Merged
merged 2 commits into from
Mar 20, 2019
Merged

Verify that rust version is recent enough to install rustlings. #131

merged 2 commits into from
Mar 20, 2019

Conversation

ColinPitrat
Copy link
Contributor

@ColinPitrat ColinPitrat commented Mar 20, 2019

This addresses issue #128.

I would have liked to write some tests for the vercomp function I
introduce, but there doesn't seem to be any CI setup yet?

I would have liked to write some tests for the vercomp function I
introduce, but there doesn't seem to be any CI setup yet?
@komaeda
Copy link
Contributor

komaeda commented Mar 20, 2019

@ColinPitrat yes, CI setup is something i'm looking at doing this evening actually! i'll let you know when

@komaeda komaeda added C-enhancement Category: Enhancement A-installation Area: Installation S-waiting-on-review Status: Waiting for review labels Mar 20, 2019
@komaeda komaeda removed the S-waiting-on-review Status: Waiting for review label Mar 20, 2019
@komaeda komaeda merged commit 91dce31 into rust-lang:master Mar 20, 2019
pedantic79 pushed a commit to pedantic79/rustlings that referenced this pull request Apr 11, 2020
Verify that rust version is recent enough to install rustlings.
ppp3 pushed a commit to ppp3/rustlings that referenced this pull request May 23, 2022
Verify that rust version is recent enough to install rustlings.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-installation Area: Installation C-enhancement Category: Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants