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

Ignore inline HTML and HTML blocks #1869

Merged
merged 4 commits into from
Jan 6, 2025
Merged

Conversation

Urgau
Copy link
Member

@Urgau Urgau commented Dec 18, 2024

This PR adds inline HTML (<a>) and HTML blocks to the ignore logic when parsing commands.

This is particularly useful for rust-lang/rust pull request template which currently hacks around triagebot finding r? in the HTML comment.

This PR also updates pulldown-cmark to 0.12.

Related to #1832
r? @ehuss

Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Thanks!

@ehuss ehuss merged commit 9342f73 into rust-lang:master Jan 6, 2025
2 checks passed
ehuss added a commit to rust-lang/rust-forge that referenced this pull request Jan 6, 2025
Urgau added a commit to Urgau/rust that referenced this pull request Jan 7, 2025
as triagebot/rustbot now ignores HTML blocks.

cf. rust-lang/triagebot#1869
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jan 7, 2025
Remove workaround from pull request template

This PR removes the workaround (`\`) from our pull request template as triagebot/rustbot now ignores HTML blocks.

cf. rust-lang/triagebot#1869
cc `@jieyouxu`
r? `@ehuss`
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Jan 7, 2025
Rollup merge of rust-lang#135189 - Urgau:triagebot-html-ignored, r=lqd

Remove workaround from pull request template

This PR removes the workaround (`\`) from our pull request template as triagebot/rustbot now ignores HTML blocks.

cf. rust-lang/triagebot#1869
cc `@jieyouxu`
r? `@ehuss`
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