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

_cargo: Make function style consistent #7060

Merged
merged 3 commits into from
Jun 22, 2019

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Jun 22, 2019

They are mostly white-space changes. I'm planning to bring back Zsh completion
for -p (package) option, which was disabled in #1713. This PR is the first
stepping stone towards that goal.

A disadvantageous is that I am just starting to learn Zsh completion system.
Guess how far can I go?

r? @ehuss

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 22, 2019
@ehuss
Copy link
Contributor

ehuss commented Jun 22, 2019

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 22, 2019

📌 Commit 44f782c has been approved by ehuss

@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 Jun 22, 2019
@bors
Copy link
Collaborator

bors commented Jun 22, 2019

⌛ Testing commit 44f782c with merge ecc41f4...

bors added a commit that referenced this pull request Jun 22, 2019
_cargo: Make function style consistent

They are just white-space changes. I'm planning to bring back Zsh completion
for `-p` (package) option, which was disabled in #1713. This PR is the first
stepping stone towards that goal.

A disadvantageous is that I am just starting to learn Zsh completion system.
Guess how far can I go?

r? @ehuss
@tesuji
Copy link
Contributor Author

tesuji commented Jun 22, 2019

@ehuss Have you reviewed 2 commits I just push? I think you might accidentally skip them.

@ehuss
Copy link
Contributor

ehuss commented Jun 22, 2019

I'm not sure what you mean. I see 2 extra commits added 3 hours ago. Is there something wrong with them? Or were you intending to add more?

@bors
Copy link
Collaborator

bors commented Jun 22, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: ehuss
Pushing ecc41f4 to master...

@bors bors merged commit 44f782c into rust-lang:master Jun 22, 2019
@tesuji tesuji deleted the zsh-complete-package branch June 22, 2019 18:40
@tesuji
Copy link
Contributor Author

tesuji commented Jun 22, 2019

No, I didn't. GitHub displayed 2 extra commits adding 3 hours ago is a lie.
I did some rebases on local and pushed it after I was satisfied with the change.
Anyway, never mind if you approved this PR.

@ehuss
Copy link
Contributor

ehuss commented Jun 22, 2019

Ah, no worries. :)

bors added a commit to rust-lang/rust that referenced this pull request Jun 24, 2019
Update cargo

17 commits in 807429e1b6da4e2ec52488ef2f59e77068c31e1f..4c1fa54d10f58d69ac9ff55be68e1b1c25ecb816
2019-06-11 14:06:10 +0000 to 2019-06-24 11:24:18 +0000
- Fix typo in comment (rust-lang/cargo#7066)
- travis: enforce formatting of subcrates as well (rust-lang/cargo#7063)
- _cargo: Make function style consistent (rust-lang/cargo#7060)
- Update some fix comments. (rust-lang/cargo#7061)
- Stabilize default-run (rust-lang/cargo#7056)
- Fix typo in comment (rust-lang/cargo#7054)
- fix(fingerpring): do not touch intermediate artifacts (rust-lang/cargo#7050)
- Resolver test/debug cleanup (rust-lang/cargo#7045)
- Rename to_url → into_url (rust-lang/cargo#7048)
- Remove needless lifetimes (rust-lang/cargo#7047)
- Revert test directory cleaning change. (rust-lang/cargo#7042)
- cargo book /reference/manifest: fix typo (rust-lang/cargo#7041)
- Extract resolver tests to their own crate (rust-lang/cargo#7011)
- ci: Do not install addons on rustfmt build jobs (rust-lang/cargo#7038)
- Support absolute paths in dep-info files (rust-lang/cargo#7030)
- ci: Run cargo fmt on all workspaces (rust-lang/cargo#7033)
- Deprecated ONCE_INIT in favor of Once::new() (rust-lang/cargo#7031)
@ehuss ehuss added this to the 1.37.0 milestone Feb 6, 2022
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.

4 participants