Skip to content
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

nits: Cleanup of librustdoc::clean::Cfg::simplify_with #134703

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

poliorcetics
Copy link
Contributor

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Dec 23, 2024
@GuillaumeGomez
Copy link
Member

Thanks! Once CI is green, r=me.

@bors delegate=poliorcetics

@bors
Copy link
Contributor

bors commented Dec 23, 2024

✌️ @poliorcetics, you can now approve this pull request!

If @GuillaumeGomez told you to "r=me" after making some further change, please make that change, then do @bors r=@GuillaumeGomez

@GuillaumeGomez
Copy link
Member

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Dec 24, 2024

📌 Commit 8cf09c7 has been approved by GuillaumeGomez

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 Dec 24, 2024
@bors
Copy link
Contributor

bors commented Dec 24, 2024

⌛ Testing commit 8cf09c7 with merge dd843fa...

bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 24, 2024
…=GuillaumeGomez

nits: Cleanup of `librustdoc::clean::Cfg::simplify_with`

r? `@GuillaumeGomez`
@rust-log-analyzer
Copy link
Collaborator

The job x86_64-msvc-ext1 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@bors
Copy link
Contributor

bors commented Dec 24, 2024

💔 Test failed - checks-actions

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

Failure was:


---- package::workspace_with_local_dep_already_published stdout ----
running `D:\a\rust\rust\build\x86_64-pc-windows-msvc\stage2-tools\x86_64-pc-windows-msvc\release\cargo.exe package`
thread 'package::workspace_with_local_dep_already_published' panicked at src\tools\cargo\tests\testsuite\package.rs:6200:10:

test failed running `D:\a\rust\rust\build\x86_64-pc-windows-msvc\stage2-tools\x86_64-pc-windows-msvc\release\cargo.exe package`
error: process exited with code 101 (expected 0)
--- stdout

--- stderr
   Packaging dep v0.1.0 (D:\a\rust\rust\build\x86_64-pc-windows-msvc\stage2-tools\x86_64-pc-windows-msvc\tmp\cit\t2250\foo\dep)
    Packaged 4 files, 1.1KiB (723.0B compressed)
   Packaging main v0.0.1 (D:\a\rust\rust\build\x86_64-pc-windows-msvc\stage2-tools\x86_64-pc-windows-msvc\tmp\cit\t2250\foo\main)
    Updating crates.io index
    Packaged 4 files, 1.5KiB (897.0B compressed)
   Verifying dep v0.1.0 (D:\a\rust\rust\build\x86_64-pc-windows-msvc\stage2-tools\x86_64-pc-windows-msvc\tmp\cit\t2250\foo\dep)
   Compiling dep v0.1.0 (D:\a\rust\rust\build\x86_64-pc-windows-msvc\stage2-tools\x86_64-pc-windows-msvc\tmp\cit\t2250\foo\target\package\dep-0.1.0)
error: couldn't create a temp dir: Access is denied. (os error 5) at path "D:\\a\\_temp\\msys64\\tmp\\rustcyEAdB6"

error: could not compile `dep` (lib) due to 1 previous error
error: failed to verify package tarball



failures:
    package::workspace_with_local_dep_already_published

test result: FAILED. 3305 passed; 1 failed; 246 ignored; 0 measured; 0 filtered out; finished in 473.07s

Completely unrelated to my changes I think ?

@GuillaumeGomez
Copy link
Member

You could have retried, you have the rights too on this PR normally. ;)

@bors retry

@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 Dec 25, 2024
@jieyouxu jieyouxu added O-windows-msvc Toolchain: MSVC, Operating system: Windows CI-spurious-fail-msvc CI spurious failure: target env msvc and removed O-windows-msvc Toolchain: MSVC, Operating system: Windows labels Dec 25, 2024
@bors
Copy link
Contributor

bors commented Dec 25, 2024

⌛ Testing commit 8cf09c7 with merge 7c002ff...

@bors
Copy link
Contributor

bors commented Dec 25, 2024

☀️ Test successful - checks-actions
Approved by: GuillaumeGomez
Pushing 7c002ff to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Dec 25, 2024
@bors bors merged commit 7c002ff into rust-lang:master Dec 25, 2024
7 checks passed
@rustbot rustbot added this to the 1.85.0 milestone Dec 25, 2024
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (7c002ff): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

This benchmark run did not return any relevant results for this metric.

Cycles

Results (primary 2.0%, secondary 3.7%)

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
2.0% [1.9%, 2.0%] 2
Regressions ❌
(secondary)
3.7% [2.8%, 4.9%] 7
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) 2.0% [1.9%, 2.0%] 2

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 764.158s -> 764.604s (0.06%)
Artifact size: 330.54 MiB -> 330.49 MiB (-0.02%)

@poliorcetics poliorcetics deleted the ab/push-ovsylkzsoxku branch December 26, 2024 20:54
poliorcetics pushed a commit to poliorcetics/rust that referenced this pull request Dec 26, 2024
…=GuillaumeGomez

nits: Cleanup of `librustdoc::clean::Cfg::simplify_with`

r? `@GuillaumeGomez`
poliorcetics pushed a commit to poliorcetics/rust that referenced this pull request Dec 28, 2024
…=GuillaumeGomez

nits: Cleanup of `librustdoc::clean::Cfg::simplify_with`

r? `@GuillaumeGomez`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-spurious-fail-msvc CI spurious failure: target env msvc merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants