Skip to content

compile rust-anaylzer with x check if it's enabled #115111

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
Aug 27, 2023

Conversation

onur-ozkan
Copy link
Member

By default, x check doesn't compile the rust-analyzer. But when it's enabled in the config's tools section, there's no reason not to do it. This change allows x check to compile rust-analyzer if it's enabled in config's tools section.

Helps to #115031

@rustbot
Copy link
Collaborator

rustbot commented Aug 22, 2023

r? @Mark-Simulacrum

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Aug 22, 2023
@onur-ozkan onur-ozkan force-pushed the check-rust-analyzer-if-enabled branch from 30833bb to f601252 Compare August 22, 2023 19:06
By default, `x check` doesn't compile the rust-analyzer. But when it's enabled in the config's
tools section, there's no reason not to do it. This change allows `x check` to compile rust-analyzer
if it's enabled in config's tools section.

Signed-off-by: ozkanonur <work@onurozkan.dev>
@onur-ozkan onur-ozkan force-pushed the check-rust-analyzer-if-enabled branch from f601252 to 76c2316 Compare August 22, 2023 19:08
@Mark-Simulacrum
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Aug 26, 2023

📌 Commit 76c2316 has been approved by Mark-Simulacrum

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 Aug 26, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2023
…iaskrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#114924 (coverage: Tidy up `run-coverage` tests in several small ways)
 - rust-lang#114927 (CI: add more debug logging to Docker caching)
 - rust-lang#114957 (tests: Fix tests for LoongArch64)
 - rust-lang#115007 (Correct and expand documentation of `handle_alloc_error` and `set_alloc_error_hook`.)
 - rust-lang#115098 (rust-gdbgui: remove excessive quotes)
 - rust-lang#115111 (compile rust-anaylzer with `x check` if it's enabled)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit ab1123b into rust-lang:master Aug 27, 2023
@rustbot rustbot added this to the 1.74.0 milestone Aug 27, 2023
@onur-ozkan onur-ozkan deleted the check-rust-analyzer-if-enabled branch August 27, 2023 14:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants