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

Move some generic test stuff out of the rust-analyzer crate #15491

Open
Veykril opened this issue Aug 21, 2023 · 0 comments
Open

Move some generic test stuff out of the rust-analyzer crate #15491

Veykril opened this issue Aug 21, 2023 · 0 comments
Labels
A-infra CI and workflow issues C-enhancement Category: enhancement

Comments

@Veykril
Copy link
Member

Veykril commented Aug 21, 2023

Currently regenerating the config, docs and some other stuff require building and running the rust-analyzer crate tests. This is annoying for people contributing fixes just in those places because they'll have to build the entire project and also figure out that they even have to do this in the first place. Additionally, if tidy fails, it will do so after having run all tests on CI which means you are effectively waiting minutes for CI only to realize you forgot a dbg! somewhere later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-infra CI and workflow issues C-enhancement Category: enhancement
Projects
None yet
Development

No branches or pull requests

1 participant