Skip to content

Replace 'project' with 'package' in many strings and comments. #6065

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

Merged
merged 2 commits into from
Sep 22, 2018

Conversation

zachlute
Copy link
Contributor

Partial fix for #6056.

Shouldn't be anything too interesting or surprising in here.

Still need to do documentation, but will do that as a different PR.

@rust-highfive
Copy link

r? @matklad

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

Copy link
Member

@dwijnand dwijnand left a comment

Choose a reason for hiding this comment

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

There was nothing to me that you changed to "package" that obviously should remain "project", so this LGTM!

Thanks for doing the man pages and completions files too!

(I pushed a test fix)

@zachlute
Copy link
Contributor Author

Ah, thanks for the test fix. Missed it because it was non-Windows and 'project' appears about a trillion times in the testsuite. :)

@alexcrichton
Copy link
Member

@bors: r=dwijnand

@bors
Copy link
Contributor

bors commented Sep 21, 2018

📌 Commit f3bd19f has been approved by dwijnand

@alexcrichton alexcrichton reopened this Sep 21, 2018
@alexcrichton
Copy link
Member

@bors: r=dwijnand

@bors
Copy link
Contributor

bors commented Sep 21, 2018

💡 This pull request was already approved, no need to approve it again.

@bors
Copy link
Contributor

bors commented Sep 21, 2018

📌 Commit f3bd19f has been approved by dwijnand

@bors
Copy link
Contributor

bors commented Sep 21, 2018

⌛ Testing commit f3bd19f with merge 18c8c5f...

bors added a commit that referenced this pull request Sep 21, 2018
Replace 'project' with 'package' in many strings and comments.

Partial fix for #6056.

Shouldn't be anything too interesting or surprising in here.

Still need to do documentation, but will do that as a different PR.
@bors
Copy link
Contributor

bors commented Sep 22, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: dwijnand
Pushing 18c8c5f to master...

@bors bors merged commit f3bd19f into rust-lang:master Sep 22, 2018
@zachlute zachlute deleted the change-project-to-package branch September 22, 2018 01:17
@ehuss ehuss added this to the 1.31.0 milestone Feb 6, 2022
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.

7 participants