Skip to content
This repository was archived by the owner on Jan 18, 2025. It is now read-only.

Conversation

@henry40408
Copy link
Contributor

No description provided.

@weihanglo
Copy link
Owner

@henry40408
I prefer to keep our main README concise and not to mention contribution details there.
Our goal is providing tutorials about common algorithms in Rust. Hence, I think separating contribution details into different document is more appropriate.

Would you mind move these great resources into another file?

@henry40408
Copy link
Contributor Author

@weihanglo No, but how we call that file? I suggest GETTING_STARTED.md. By the way, I insist to put a link to the file in README.md. We can't expect everyone to have Rust already installed on their laptop.

@weihanglo
Copy link
Owner

I am fine with GETTING_STARTED.md. We can include the link to GETTING_STARTED.md instead of the whole details.

@henry40408
Copy link
Contributor Author

@weihanglo updated, please review

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe override directory toolchain would be a better choice.

rustup override set stable

@weihanglo
Copy link
Owner

I'll merge this first. We can send following PRs later.

@weihanglo weihanglo merged commit 011bbd0 into weihanglo:master Dec 29, 2018
@weihanglo weihanglo deleted the feature/doc branch December 29, 2018 01:42
@weihanglo weihanglo restored the feature/doc branch December 29, 2018 01:43
@henry40408 henry40408 deleted the feature/doc branch December 29, 2018 23:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants