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

Book: in beginner guide change irc channel #rust → #rust-beginners #32622

Merged
merged 4 commits into from
Apr 2, 2016
Merged

Book: in beginner guide change irc channel #rust → #rust-beginners #32622

merged 4 commits into from
Apr 2, 2016

Conversation

tyoc213
Copy link
Contributor

@tyoc213 tyoc213 commented Mar 30, 2016

#32585 r? @alexcrichton

I also would like to add the reference on the first README.md

Some like

 most popular channel is [#rust], a venue for general discussion about
-Rust, and a good place to ask for help.
+Rust. And a good place to ask for help would be [#rust-beginners].

 [IRC]: https://en.wikipedia.org/wiki/Internet_Relay_Chat
 [#rust]: irc://irc.mozilla.org/rust
+[#rust-beginners]: irc://irc.mozilla.org/rust-beginners

So In the first page would be the two options for #rust or #rust-beginners

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@steveklabnik
Copy link
Member

👍 from me

@tyoc213
Copy link
Contributor Author

tyoc213 commented Mar 30, 2016

So, I will add the change to the README.md or it should only be the change on the book?

If it where the change to the readme, the text in the first comment would be the correct wording?

@alexcrichton
Copy link
Member

Sounds good to me to change the README as well! Perhaps the #rust channel could also still be mentioned here along with #rust-beginners?

@tyoc213
Copy link
Contributor Author

tyoc213 commented Mar 31, 2016

Just added the info to the README and updated the part in getting-started for #rust and #rust-beginners

@@ -164,11 +164,15 @@ installed. Doing so will depend on your specific system, consult its
documentation for more details.

If not, there are a number of places where we can get help. The easiest is
[the #rust-beginners IRC channel on irc.mozilla.org][irc-beginners], which we
can access through [Mibbit][mibbit-beginners] and for general discussion
[the #rust IRC channel on irc.mozilla.org][irc], which we can access through
Copy link
Member

Choose a reason for hiding this comment

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

This second clause of "which we can access through mibbit" can probably be dropped as it was already mentioned before, otherwise looks good to me!

@tyoc213
Copy link
Contributor Author

tyoc213 commented Mar 31, 2016

Well, the only problem I see now, is that I see only my last commit as the git user here, should I replace my user in those commits? with something like change email? or should I leave it like that?


Well I didn't use the method above, but instead git rebase -i HEAD~4 and amend the author, should I force the push on my branch so the commits have the correct user email?

@steveklabnik
Copy link
Member

@tyoc213 yeah, just force push those

@tyoc213
Copy link
Contributor Author

tyoc213 commented Mar 31, 2016

Done, thanks!

@alexcrichton
Copy link
Member

@bors: r+ 9094935

@steveklabnik
Copy link
Member

@bors: rollup

On Mar 31, 2016, 10:32 -0700, Alex [email protected], wrote:

@bors(https://github.com/bors): r+9094935(9094935)


You are receiving this because you commented.
Reply to this email directly orview it on GitHub(#32622 (comment))

Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 1, 2016
Book: in beginner guide change irc channel #rust → #rust-beginners

I also would like to add the reference on the first README.md

Some like

```
 most popular channel is [#rust], a venue for general discussion about
-Rust, and a good place to ask for help.
+Rust. And a good place to ask for help would be [#rust-beginners].

 [IRC]: https://en.wikipedia.org/wiki/Internet_Relay_Chat
 [#rust]: irc://irc.mozilla.org/rust
+[#rust-beginners]: irc://irc.mozilla.org/rust-beginners
```

So In the first page would be the two options for #rust or #rust-beginners
bors added a commit that referenced this pull request Apr 2, 2016
Rollup of 11 pull requests

- Successful merges: #32622, #32629, #32640, #32641, #32642, #32645, #32647, #32649, #32652, #32654, #32656
- Failed merges:
@bors bors merged commit 9094935 into rust-lang:master Apr 2, 2016
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.

5 participants