Skip to content

Commit d45a79f

Browse files
authored
Merge pull request #1914 from Urgau/imprv-mcp-opening-comment
Improve major change opening comment
2 parents fd32535 + 39fafda commit d45a79f

File tree

1 file changed

+23
-29
lines changed

1 file changed

+23
-29
lines changed

src/handlers/major_change.rs

+23-29
Original file line numberDiff line numberDiff line change
@@ -259,35 +259,29 @@ async fn handle(
259259
if new_proposal {
260260
let topic_url = zulip_req.url();
261261
let comment = format!(
262-
"> [!IMPORTANT] \n\
263-
> This issue is *not meant to be used for technical discussion*. \
264-
There is a **Zulip [stream]** for that. Use this issue to leave \
265-
procedural comments, such as volunteering to review, indicating that you \
266-
second the proposal (or third, etc), or raising a concern that you would \
267-
like to be addressed. \
268-
\n\n \
269-
Concerns or objections to the proposal should be discussed on Zulip and formally registered \
270-
here by adding a comment with the following syntax: \
271-
\n \
272-
``` \
273-
\n \
274-
@rfcbot concern reason-for-concern \
275-
\n \
276-
<description of the concern> \
277-
\n \
278-
``` \
279-
\n \
280-
Concerns can be lifted with: \
281-
\n \
282-
``` \
283-
\n \
284-
@rfcbot resolve reason-for-concern \
285-
\n \
286-
``` \
287-
\n\n \
288-
See documentation at [https://forge.rust-lang.org](https://forge.rust-lang.org/compiler/mcp.html#what-kinds-of-comments-should-go-on-the-tracking-issue-in-compiler-team-repo) \
289-
\n\n{} \
290-
\n\n[stream]: {}",
262+
r#"> [!IMPORTANT]
263+
> This issue is *not meant to be used for technical discussion*. There is a **Zulip [stream]** for that.
264+
> Use this issue to leave procedural comments, such as volunteering to review, indicating that you second the proposal (or third, etc), or raising a concern that you would like to be addressed.
265+
266+
<details>
267+
<summary>Concerns or objections can formally be registered here by adding a comment.</summary>
268+
<p>
269+
270+
```
271+
@rfcbot concern reason-for-concern
272+
<description of the concern>
273+
```
274+
Concerns can be lifted with:
275+
```
276+
@rfcbot resolve reason-for-concern
277+
```
278+
See documentation at [https://forge.rust-lang.org](https://forge.rust-lang.org/compiler/mcp.html#what-kinds-of-comments-should-go-on-the-tracking-issue-in-compiler-team-repo)
279+
280+
</p>
281+
</details>
282+
{}
283+
284+
[stream]: {}"#,
291285
config.open_extra_text.as_deref().unwrap_or_default(),
292286
topic_url
293287
);

0 commit comments

Comments
 (0)