Skip to content

Conversation

carols10cents
Copy link
Member

reassignment is the correct spelling rather than re-assignment; this error message looks silly in the book next to text trying to be grammatically correct :-/

Will this cause any stability/backcompat type issues?

@carols10cents carols10cents added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 20, 2017
@rust-highfive
Copy link
Contributor

r? @arielb1

(rust_highfive has picked a reviewer for you, use r? to override)

@carols10cents
Copy link
Member Author

carols10cents commented Oct 23, 2017

pinging on my own PR but it's what the triage procedures say sooo ping @arielb1! pinging you on irc too! or not because you're not online :)

@arielb1
Copy link
Contributor

arielb1 commented Oct 23, 2017

This won't cause any stability problems - there are no stability promises on error messages.

However, I'm not sure this is an improvement. assignment is the technical term, so I think re-assignment is preferrable to the somewhat distinct word reassignment.

@kennytm
Copy link
Member

kennytm commented Oct 25, 2017

So, should this PR go forward or should this be closed?

@carols10cents
Copy link
Member Author

I think we've come up with a compromise :)

@arielb1
Copy link
Contributor

arielb1 commented Oct 25, 2017

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 25, 2017

📌 Commit 0e46cf4 has been approved by arielb1

@kennytm kennytm 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 25, 2017
kennytm added a commit to kennytm/rust that referenced this pull request Oct 25, 2017
Correct misspelling in error text: re-assignment => reassignment

[reassignment is the correct spelling](https://www.thefreedictionary.com/reassignment) rather than re-assignment; this error message looks silly in the book next to text trying to be grammatically correct :-/

Will this cause any stability/backcompat type issues?
bors added a commit that referenced this pull request Oct 25, 2017
Rollup of 7 pull requests

- Successful merges: #45059, #45212, #45398, #45483, #45496, #45508, #45526
- Failed merges:
@bors bors merged commit 0e46cf4 into rust-lang:master Oct 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants