Skip to content

Revert pull_request field change. #1653

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

Merged
merged 1 commit into from
Sep 11, 2022

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