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

Allow rust-project.json to be hidden #17818

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

alibektas
Copy link
Member

Closes #17816

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 7, 2024
@Veykril
Copy link
Member

Veykril commented Aug 7, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 7, 2024

📌 Commit 2426649 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Aug 7, 2024

⌛ Testing commit 2426649 with merge ddcd66b...

@bors
Copy link
Collaborator

bors commented Aug 7, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing ddcd66b to master...

@bors bors merged commit ddcd66b into rust-lang:master Aug 7, 2024
11 checks passed
@davidbarsky
Copy link
Contributor

Thanks for doing this! I'm pretty sure this was the last thing I needed before using it in my employer's monorepo.

intel-lab-lkp pushed a commit to intel-lab-lkp/linux that referenced this pull request Aug 17, 2024
Very soon after we requested it [1], `rust-analyzer` added support for
`.rust-project.json` [2], i.e. the hidden variant of `.rust-project.json`.

While this is only a (tiny) issue for kernel developers that use Rust
(i.e. so far a very small number compared to the total), the file is not
really something most kernel developers need to see, care or modify while
working, so it is not really needed to show it by default in listings.

It also follows the pattern of other files that are not directly related
to the project (unlike e.g. a `Kconfig` file).

Thus migrate to it.

Cc: Ali Bektas <bektasali@protonmail.com>
Cc: Lukas Wirth <lukastw97@gmail.com>
Link: rust-lang/rust-analyzer#17816 [1]
Link: rust-lang/rust-analyzer#17818 [2]
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support .rust-project.json (i.e. hidden)
5 participants