Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Sep 11, 2022

The change to the pull_request field in #1646 is causing way more trouble than it is worth. If GitHub sometimes populates this field, then it is useful to know if it is a pull request or not.

Fixes #1651.

@Mark-Simulacrum Mark-Simulacrum merged commit 6ba29f7 into rust-lang:master Sep 11, 2022
@apiraino
Copy link
Contributor

For future notice, should this codepath will be changed: #1646 also was breaking the T-compiler agenda generation (specifically commit 3cde23d) when running the triagebot command cargo run --bin prioritization-agenda.

Just mentioning this because some dependencies are not always apparent with this codebase. A good strategy could be to run all three CLI binaries of the triagebot (compiler,lang,prioritization-agenda) for a quick sanity check.

Anyway, no problem, this patch reverts the breaking change :)

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.

@rustbot ready and @rustbot author erroring out in a few cases

3 participants