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

Advent of Code rust #46

Merged
merged 2 commits into from Apr 1, 2016
Merged

Advent of Code rust #46

merged 2 commits into from Apr 1, 2016

Conversation

LD250
Copy link
Contributor

@LD250 LD250 commented Mar 24, 2016

Hello @ctjhoa ,
Now I have work in progress of finishing http://adventofcode.com/ puzzles using rust.
I have a plan to do all the puzzles from 11 through 25.
Not sure if I am using "rust best practice" there, but maybe this will come in handy for Rust learners.

@ctjhoa
Copy link
Owner

ctjhoa commented Mar 25, 2016

Thanks @LD250
I haven't test it yet but it seems cover programming in general more than rust specifically.
Am I right ?

@LD250
Copy link
Contributor Author

LD250 commented Mar 25, 2016

Yes. Its a general programming problems solved with rust.
I saw something similar in your repo, Project Euler (https://github.com/gifnksm/ProjectEulerRust).

So if the Project Euler solved with rust is good to be shown here, maybe the Advent of Code solved with rust will be good to show too.

@LD250
Copy link
Contributor Author

LD250 commented Mar 25, 2016

Here is the the link to the repo for quick access https://github.com/LD250/adventofcode_rust

@ctjhoa
Copy link
Owner

ctjhoa commented Apr 1, 2016

@LD250 sorry I've been really busy. Seems ok to merge to me.

Hint: Your code doesn't seems to be idiomatic rust code, maybe try to use rustfmt and/or to post it on rust forum or reddit for improvement.
Anyway, thanks!

@ctjhoa ctjhoa merged commit 929d806 into ctjhoa:master Apr 1, 2016
@LD250
Copy link
Contributor Author

LD250 commented Apr 1, 2016

@ctjhoa thank you for the advise. I will try rustfmt.

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.

None yet

2 participants