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

doc_lazy_continuation: do not warn on End events #12818

Merged
merged 1 commit into from
May 18, 2024

Conversation

notriddle
Copy link
Contributor

changelog: none

This avoids event spans that would otherwise cause crashes, since an
End's span covers the range of the tag (which will be earlier than the
line break within the tag).

@rustbot
Copy link
Collaborator

rustbot commented May 18, 2024

r? @y21

rustbot has assigned @y21.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label May 18, 2024
Copy link
Member

@y21 y21 left a comment

Choose a reason for hiding this comment

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

Looks good, r=me with rustfmt fixed

@y21
Copy link
Member

y21 commented May 18, 2024

cc @flip1995 , this should unblock the sync

This avoids event spans that would otherwise cause crashes, since an
End's span covers the range of the tag (which will be earlier than the
line break within the tag).
@notriddle
Copy link
Contributor Author

Fixed the rustfmt complaint. I didn't want to actually reformat the markdown, because I wanted it to match the one in fuchsia.

@y21
Copy link
Member

y21 commented May 18, 2024

Yeah that seems fine. Thanks!

@bors r+

@bors
Copy link
Collaborator

bors commented May 18, 2024

📌 Commit b5cf8b8 has been approved by y21

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented May 18, 2024

⌛ Testing commit b5cf8b8 with merge 0b1bf37...

@bors
Copy link
Collaborator

bors commented May 18, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: y21
Pushing 0b1bf37 to master...

@bors bors merged commit 0b1bf37 into rust-lang:master May 18, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants