Skip to content

Fix post-merge workflow #138335

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 11, 2025
Merged

Fix post-merge workflow #138335

merged 1 commit into from
Mar 11, 2025

Conversation

jieyouxu
Copy link
Member

@jieyouxu jieyouxu commented Mar 11, 2025

The command is called post-merge-report not post-merge-analysis. See

PostMergeReport {

Noticed it failing in #138310 (comment).

r? @Kobzol (or @marcoieni)

The command is called `post-merge-report` not `post-merge-analysis`.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 11, 2025
@jieyouxu jieyouxu added T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. A-CI Area: Our Github Actions CI labels Mar 11, 2025
@marcoieni
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Mar 11, 2025

📌 Commit c00a5c0 has been approved by marcoieni

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 11, 2025
@Kobzol
Copy link
Contributor

Kobzol commented Mar 11, 2025

Thanks, you beat me to it. This was embarassing 🤦

@jieyouxu
Copy link
Member Author

Happens 😁

Kobzol added a commit to Kobzol/rust that referenced this pull request Mar 11, 2025
Fix post-merge workflow

The command is called `post-merge-report` not `post-merge-analysis`. See https://github.com/rust-lang/rust/blob/90384941aae4ea38de00e4702b50757e9b882a19/src/ci/citool/src/main.rs#L379

Noticed it failing in rust-lang#138310 (comment).

r? `@Kobzol` (or `@marcoieni)`
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 11, 2025
Rollup of 11 pull requests

Successful merges:

 - rust-lang#135987 (Clarify iterator by_ref docs)
 - rust-lang#137967 ([AIX] Fix hangs during testing)
 - rust-lang#138063 (Improve `-Zunpretty=hir` for parsed attrs)
 - rust-lang#138147 (Add maintainers for powerpc64le-unknown-linux-gnu)
 - rust-lang#138288 (Document -Z crate-attr)
 - rust-lang#138300 (add tracking issue for unqualified_local_imports)
 - rust-lang#138307 (Allow specifying glob patterns for try jobs)
 - rust-lang#138315 (use next_back() instead of last() on DoubleEndedIterator)
 - rust-lang#138330 (Remove unnecessary `[lints.rust]` sections.)
 - rust-lang#138335 (Fix post-merge workflow)
 - rust-lang#138343 (Enable `f16` tests for `powf`)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 11, 2025
Rollup of 10 pull requests

Successful merges:

 - rust-lang#135987 (Clarify iterator by_ref docs)
 - rust-lang#137967 ([AIX] Fix hangs during testing)
 - rust-lang#138063 (Improve `-Zunpretty=hir` for parsed attrs)
 - rust-lang#138147 (Add maintainers for powerpc64le-unknown-linux-gnu)
 - rust-lang#138288 (Document -Z crate-attr)
 - rust-lang#138300 (add tracking issue for unqualified_local_imports)
 - rust-lang#138307 (Allow specifying glob patterns for try jobs)
 - rust-lang#138315 (use next_back() instead of last() on DoubleEndedIterator)
 - rust-lang#138330 (Remove unnecessary `[lints.rust]` sections.)
 - rust-lang#138335 (Fix post-merge workflow)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 64c6ec5 into rust-lang:master Mar 11, 2025
6 checks passed
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 11, 2025
Rollup merge of rust-lang#138335 - jieyouxu:fix-citool, r=marcoieni

Fix post-merge workflow

The command is called `post-merge-report` not `post-merge-analysis`. See https://github.com/rust-lang/rust/blob/90384941aae4ea38de00e4702b50757e9b882a19/src/ci/citool/src/main.rs#L379

Noticed it failing in rust-lang#138310 (comment).

r? ``@Kobzol`` (or ``@marcoieni)``
@rustbot rustbot added this to the 1.87.0 milestone Mar 11, 2025
@jieyouxu jieyouxu deleted the fix-citool branch March 11, 2025 22:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-CI Area: Our Github Actions CI S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants