Skip to content

markdown: Add output assertion to the doc test #3932

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
Sep 19, 2021

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Sep 17, 2021

This PR adjusts the doc comment of the readme_to_html() function by adding an assertion for the rendered result.

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Sep 17, 2021
@Turbo87 Turbo87 requested review from 0xPoe and JohnTitor September 17, 2021 21:44
@bors
Copy link
Contributor

bors commented Sep 18, 2021

☔ The latest upstream changes (presumably #3933) made this pull request unmergeable. Please resolve the merge conflicts.

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 19, 2021

📌 Commit d40d484 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Sep 19, 2021

⌛ Testing commit d40d484 with merge ea78227...

@bors
Copy link
Contributor

bors commented Sep 19, 2021

☀️ Test successful - checks-actions
Approved by: JohnTitor
Pushing ea78227 to master...

@bors bors merged commit ea78227 into rust-lang:master Sep 19, 2021
@Turbo87 Turbo87 deleted the markdown-doc-test branch September 19, 2021 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants