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

feat(primitive_types6): Add a test #548

Merged
merged 3 commits into from Oct 10, 2020
Merged

Conversation

@darnuria
Copy link
Contributor

@darnuria darnuria commented Oct 4, 2020

To keep constancy with primitive_types4.rs I suggest to add a test in primitive_types6.rs this suggestion comes from my students because some students were just typing any numbers.

Also added a little indication in order to avoid passing solution with just typing the number. Also it's possible to type out numbers.??? to avoid this possible unexpected outcome I got it in class this week.

If it's not an issue or if this not merge-able in this state I am open to discussion! :)

@darnuria darnuria force-pushed the darnuria:primitive_types6/test branch from e0271e0 to 2919fde Oct 4, 2020
@darnuria
Copy link
Contributor Author

@darnuria darnuria commented Oct 4, 2020

Did a force push to problem description and rewrite syntax instead of syntaxe (french way to write the same word). ;)

Co-authored-by: Annika <56906084+AnnikaCodes@users.noreply.github.com>
@fmoko
fmoko approved these changes Oct 10, 2020
Copy link
Member

@fmoko fmoko left a comment

Thanks!

@fmoko fmoko changed the title Add a test in primitive_types6.rs feat(primitive_types6): Add a test Oct 10, 2020
@fmoko fmoko merged commit 2b1fb2b into rust-lang:main Oct 10, 2020
@fmoko
Copy link
Member

@fmoko fmoko commented Oct 10, 2020

@all-contributors please add @darnuria for code

@allcontributors
Copy link
Contributor

@allcontributors allcontributors bot commented Oct 10, 2020

@fmoko

I've put up a pull request to add @darnuria! 🎉

@darnuria darnuria deleted the darnuria:primitive_types6/test branch Oct 10, 2020
@fmoko
Copy link
Member

@fmoko fmoko commented Oct 10, 2020

@all-contributors please add @AnnikaCodes for review

@allcontributors
Copy link
Contributor

@allcontributors allcontributors bot commented Oct 10, 2020

@fmoko

I've put up a pull request to add @AnnikaCodes! 🎉

@fmoko
Copy link
Member

@fmoko fmoko commented Oct 10, 2020

let's try this again without conflicts

@all-contributors please add @AnnikaCodes for review

@allcontributors
Copy link
Contributor

@allcontributors allcontributors bot commented Oct 10, 2020

@fmoko

I've put up a pull request to add @AnnikaCodes! 🎉

@fmoko
Copy link
Member

@fmoko fmoko commented Oct 10, 2020

@all-contributors please add @AnnikaCodes for review

@allcontributors
Copy link
Contributor

@allcontributors allcontributors bot commented Oct 10, 2020

@fmoko

I've put up a pull request to add @AnnikaCodes! 🎉

catull added a commit to catull/rustlings that referenced this pull request Oct 24, 2020
Co-authored-by: Annika <56906084+AnnikaCodes@users.noreply.github.com>
Co-authored-by: fmoko <mokou@posteo.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.