Skip to content

Conversation

tshepang
Copy link
Member

Closes #2261

src/tests/ui.md Outdated
Comment on lines 418 to 422
This directive takes comma-separated issue numbers as arguments,
each prefixed with a `#`.
Examples are `//@ known-bug: #12345` and `//@ known-bug: #123, #456`.
Arbitrary text before the `#` is also accepted,
example being `rust-lang/rust#12345`.
Copy link
Member

Choose a reason for hiding this comment

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

Suggestion: probably sth like

The known-bug directive accepts issue numbers or "unknown". Valid forms:

  • //@ known-bug: #123456 (when the issue is on rust-lang/rust)
  • //@ known-bug: rust-lang/chalk#123456 (when the issue is on another repo)
  • //@ known-bug: unknown (when there is no known issue yet; preferrably open one if it does not already exist)

@jieyouxu jieyouxu added S-waiting-on-author Status: this PR is waiting for additional action by the OP T-bootstrap Relevant to bootstrap team labels Mar 17, 2025
@jieyouxu jieyouxu self-assigned this Mar 17, 2025
@tshepang tshepang changed the title mention that know-bug test directive takes arguments mention that known-bug test directive takes arguments Mar 19, 2025
@tshepang tshepang force-pushed the tshepang-known-bug-takes-args branch from aed9cc4 to bfb70a7 Compare March 29, 2025 18:01
@tshepang tshepang force-pushed the tshepang-known-bug-takes-args branch from bfb70a7 to 6c1077e Compare March 29, 2025 18:02
Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Thanks

@jieyouxu jieyouxu merged commit fdca2d3 into master Mar 29, 2025
1 check passed
@tshepang tshepang deleted the tshepang-known-bug-takes-args branch March 29, 2025 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: this PR is waiting for additional action by the OP T-bootstrap Relevant to bootstrap team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

known-bug directive syntax is not documented
2 participants