Skip to content

Document IndexMatchedPath #24

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 1 commit into from
Dec 8, 2014

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Dec 6, 2014

Since rust-lang/rust#19553, missing_docs checks typedefs too.

Since rust-lang/rust#19553, missing_docs checks typedefs too.
@cuviper
Copy link
Member Author

cuviper commented Dec 6, 2014

The travis failure looks unrelated -- might even be some transient issue. cargo doc works fine for me, but here's what travis said:

$ cargo doc
Compiling url v0.2.2 (https://github.com/servo/rust-url#bdda3a7b)
Compiling gcc v0.1.0 (https://github.com/alexcrichton/gcc-rs#3caf7309)
Compiling pkg-config v0.1.0
Compiling pkg-config v0.1.0 (https://github.com/alexcrichton/pkg-config-rs#9b3b44a2)
Build failed, waiting for other jobs to finish...
Could not document `pkg-config`.
--- stderr
task '<main>' panicked at 'failed to generate documentation: couldn't create directory (File exists; path=/home/travis/build/alexcrichton/git2-rs/target/doc; mode=0700)', /home/rustbuild/src/rust-buildbot/slave/nightly-linux/build/src/librustdoc/lib.rs:258

@alexcrichton
Copy link
Member

Oops I think I pushed another fix before I saw this, sorry about that! I'll merge by hand, thanks!

@alexcrichton alexcrichton merged commit 66cce8f into rust-lang:master Dec 8, 2014
@cuviper
Copy link
Member Author

cuviper commented Dec 8, 2014

No worries - thanks!
On Dec 7, 2014 6:27 PM, "Alex Crichton" [email protected] wrote:

Merged #24 https://github.com/alexcrichton/git2-rs/pull/24.


Reply to this email directly or view it on GitHub
https://github.com/alexcrichton/git2-rs/pull/24#event-203911196.

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.

2 participants