Skip to content

Conversation

Skgland
Copy link
Contributor

@Skgland Skgland commented Aug 23, 2025

and suggest a retry if they are many

resolves #791

@Mark-Simulacrum Mark-Simulacrum added this pull request to the merge queue Aug 23, 2025
Merged via the queue into rust-lang:master with commit 5aa9e74 Aug 23, 2025
3 checks passed
@Skgland Skgland deleted the note-spurious-retry-list-entries branch August 24, 2025 09:12
res.info.values().sum::<u32>(),
),
)
.line("bar_chart", format!(" {spurious_retry} spurious results on the [retry-regessed-list.txt]({retry_regressed_list_url}), consider a retry[^1] if this is a significant amount.\n[^1]: re-run the experiment with `crates={retry_regressed_list_url}`"))
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
.line("bar_chart", format!(" {spurious_retry} spurious results on the [retry-regessed-list.txt]({retry_regressed_list_url}), consider a retry[^1] if this is a significant amount.\n[^1]: re-run the experiment with `crates={retry_regressed_list_url}`"))
.line("bar_chart", format!(" {spurious_retry} spurious results on the [retry-regressed-list.txt]({retry_regressed_list_url}), consider a retry[^1] if this is a significant amount.\n[^1]: re-run the experiment with `crates={retry_regressed_list_url}`"))
Comment

Skgland added a commit to Skgland/crater that referenced this pull request Sep 20, 2025
lqd pointed this out in rust-lang#793 (comment)
@Skgland Skgland mentioned this pull request Sep 20, 2025
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.

Should the crater-bot result comment contain the amount of crates on the retry-regressed-list.txt?

3 participants