Skip to content

Fixed docs.rs metadata#744

Merged
RCasatta merged 1 commit intorust-bitcoin:masterfrom
Kixunil:fix-docsrs
Dec 27, 2021
Merged

Fixed docs.rs metadata#744
RCasatta merged 1 commit intorust-bitcoin:masterfrom
Kixunil:fix-docsrs

Conversation

@Kixunil
Copy link
Copy Markdown
Collaborator

@Kixunil Kixunil commented Dec 17, 2021

This changes rustc-args, which doesn't do what we want, to rustdoc-args,
which does.

Because of huge value/effort ratio assigning medium priority and 0.28 milestone - would be a real shame to release without this.

This changes `rustc-args`, which doesn't do what we want, to `rustdoc-args`,
which does.
@Kixunil Kixunil added bug documentation P-medium Medium priority trivial Obvious, easy and quick to review (few lines or doc-only...) labels Dec 17, 2021
@Kixunil Kixunil added this to the 0.28.0 milestone Dec 17, 2021
@dpc
Copy link
Copy Markdown
Contributor

dpc commented Dec 17, 2021

https://docs.rs/about/metadata for reference

@Kixunil
Copy link
Copy Markdown
Collaborator Author

Kixunil commented Dec 17, 2021

@dpc the other part of it is knowing the difference between RUSTFLAGS and RUSTDOCFLAGS :)

Copy link
Copy Markdown
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

blindly ACK 9ef1c1e

I don't understand what this change does but it looks harmless.

@Kixunil
Copy link
Copy Markdown
Collaborator Author

Kixunil commented Dec 23, 2021

@apoelstra makes sure docs.rs builds docs with correct flag so that all the work on documenting features actually makes sense - they don't appear there without the flag.

Copy link
Copy Markdown
Collaborator

@RCasatta RCasatta left a comment

Choose a reason for hiding this comment

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

utACK 9ef1c1e

@RCasatta RCasatta merged commit 5f47550 into rust-bitcoin:master Dec 27, 2021
@Kixunil Kixunil deleted the fix-docsrs branch December 27, 2021 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug P-medium Medium priority trivial Obvious, easy and quick to review (few lines or doc-only...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants