Skip to content

Improve docs on some char boolean methods #65165

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from Oct 13, 2019
Merged

Improve docs on some char boolean methods #65165

merged 1 commit into from Oct 13, 2019

Conversation

ghost
Copy link

@ghost ghost commented Oct 6, 2019

simple revival of #61794
(also rustfmt on rest of file :)

Documentation for is_xid_start() and is_xid_continue() couldn't be improved since both methods got remove from this repository

r? @dtolnay
cc @JohnCSimon

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 6, 2019
@JohnCSimon
Copy link
Member

Ping from triage
@dtolnay can you please review this PR?
cc: @varkor @SimonSapin

thanks

Copy link
Member

@varkor varkor left a comment

Choose a reason for hiding this comment

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

Looks good to me. Just a couple of missing commas.

@varkor varkor assigned varkor and unassigned dtolnay Oct 12, 2019
@varkor
Copy link
Member

varkor commented Oct 12, 2019

Thanks!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 12, 2019

📌 Commit d8c2956 has been approved by varkor

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 12, 2019
Centril added a commit to Centril/rust that referenced this pull request Oct 13, 2019
Improve docs on some char boolean methods

simple revival of rust-lang#61794
(also rustfmt on rest of file :)

Documentation for `is_xid_start()` and `is_xid_continue()` couldn't be improved since both methods got remove from this repository

r? @dtolnay
cc @JohnCSimon
bors added a commit that referenced this pull request Oct 13, 2019
Rollup of 13 pull requests

Successful merges:

 - #65039 (Document missing deny by default lints)
 - #65069 (Implement Clone::clone_from for VecDeque)
 - #65165 (Improve docs on some char boolean methods)
 - #65248 (Suggest `if let` on `let` refutable binding)
 - #65250 (resolve: fix error title regarding private constructors)
 - #65295 (Move diagnostics code out of the critical path)
 - #65320 (Report `CONST_ERR` lint in external macros)
 - #65327 (replace the hand-written binary search with the library one)
 - #65339 (do not reference LLVM for our concurrency memory model)
 - #65357 (syntax: simplify maybe_annotate_with_ascription)
 - #65358 (simplify maybe_stage_features)
 - #65359 (simplify integer_lit)
 - #65360 (mbe: reduce panictry! uses.)

Failed merges:

r? @ghost
@bors bors merged commit d8c2956 into rust-lang:master Oct 13, 2019
@bors
Copy link
Collaborator

bors commented Oct 13, 2019

☔ The latest upstream changes (presumably #65368) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 13, 2019
@ghost ghost deleted the char_docs branch October 13, 2019 16:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants