Skip to content

/cargo

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

Fix test name completion in ZSH #1713

Merged
merged 5 commits into from Jun 14, 2015

Conversation

Reviewers
No reviews
Assignees

@wycats wycats

Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@eerden @rust-highfive @alexcrichton @bors @wycats
@eerden
Copy link
Contributor

eerden commented Jun 13, 2015

Fixes #1703

eerden added some commits Jun 13, 2015

@eerden
Fix error causing test name completion failure 
This was caused by a syntax error.
f952442
@eerden
Fix wrong auto completion for update --precise 
--precise argument for the update command was fed test names. This
commit prevents autocompletion of --precise.
9c812d5
@eerden
Improve autocompletion of test items 
Test item autocompletion didn't work because it required the name of the
test to come immediately after [[test]] line. This commit should fix
that.
d2c80fd
@eerden
Fix bench name completion
1ef1965
@eerden
Disable package completion
Loading status checks…
d7cf7d9

@rust-highfive rust-highfive assigned wycats Jun 13, 2015

@rust-highfive

This comment has been minimized.

Sign in to view
Copy link

rust-highfive commented Jun 13, 2015

r? @wycats

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

This comment has been minimized.

Sign in to view
Copy link
Member

alexcrichton commented Jun 14, 2015

@bors: r+ d7cf7d9
@bors

This comment has been minimized.

Sign in to view
Copy link
Contributor

bors commented Jun 14, 2015

⌛️ Testing commit d7cf7d9 with merge ba47db2...

bors added a commit that referenced this pull request Jun 14, 2015

@bors
Auto merge of #1713 - eerden:zsh-autocompletion-testnames-fix, r=alex… 
…crichton

Fixes #1703
Loading status checks…
ba47db2
@bors

This comment has been minimized.

Sign in to view
Copy link
Contributor

bors commented Jun 14, 2015

☀️ Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64

@bors bors merged commit d7cf7d9 into rust-lang:master Jun 14, 2015
2 checks passed

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.