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

fix(option1): Add test for prematurely passing exercise #198

Merged
merged 1 commit into from
Jul 27, 2019

Conversation

nkanderson
Copy link
Contributor

Fixes the bug referenced in #160, but does not address the larger feature work referenced by the issue.

Fixes the bug referenced in rust-lang#160, but does not address the larger feature work referenced by the issue.
@komaeda
Copy link
Contributor

komaeda commented Jul 27, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 27, 2019

📌 Commit a750e4a has been approved by komaeda

@bors
Copy link
Contributor

bors commented Jul 27, 2019

⌛ Testing commit a750e4a with merge 1890b00...

bors added a commit that referenced this pull request Jul 27, 2019
fix(option1): Add test for prematurely passing exercise

Fixes the bug referenced in #160, but does not address the larger feature work referenced by the issue.
@bors
Copy link
Contributor

bors commented Jul 27, 2019

☀️ Test successful - checks-travis
Approved by: komaeda
Pushing 1890b00 to master...

@bors bors merged commit a750e4a into rust-lang:master Jul 27, 2019
dylnuge added a commit to dylnuge/rustlings that referenced this pull request Aug 18, 2019
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
bors added a commit that referenced this pull request Aug 18, 2019
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in #160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to #198. It does not take a stance on
the correct way to handle this for all exercises; see #127. There are
likely other exercises whose intent are masked by this issue.
pedantic79 pushed a commit to pedantic79/rustlings that referenced this pull request Apr 11, 2020
…aeda

fix(option1): Add test for prematurely passing exercise

Fixes the bug referenced in rust-lang#160, but does not address the larger feature work referenced by the issue.
pedantic79 pushed a commit to pedantic79/rustlings that referenced this pull request Apr 11, 2020
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
pedantic79 pushed a commit to pedantic79/rustlings that referenced this pull request Apr 11, 2020
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in rust-lang#160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
ppp3 pushed a commit to ppp3/rustlings that referenced this pull request May 23, 2022
…aeda

fix(option1): Add test for prematurely passing exercise

Fixes the bug referenced in rust-lang#160, but does not address the larger feature work referenced by the issue.
ppp3 pushed a commit to ppp3/rustlings that referenced this pull request May 23, 2022
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
ppp3 pushed a commit to ppp3/rustlings that referenced this pull request May 23, 2022
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in rust-lang#160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
dmoore04 pushed a commit to dmoore04/rustlings that referenced this pull request Sep 11, 2022
…aeda

fix(option1): Add test for prematurely passing exercise

Fixes the bug referenced in rust-lang#160, but does not address the larger feature work referenced by the issue.
dmoore04 pushed a commit to dmoore04/rustlings that referenced this pull request Sep 11, 2022
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
dmoore04 pushed a commit to dmoore04/rustlings that referenced this pull request Sep 11, 2022
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in rust-lang#160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
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