Skip to content

feat: Allow unsetting env vars in server.extraEnv config #19634

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

Merged
merged 1 commit into from
Apr 21, 2025

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Apr 20, 2025

Fixes #19627

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 20, 2025
@Veykril Veykril force-pushed the push-mnpmxxrprymo branch 2 times, most recently from a3b364f to 7918812 Compare April 21, 2025 06:24
@Veykril
Copy link
Member Author

Veykril commented Apr 21, 2025

@BenjaminBrienen the typescript CI seems to fail now on the --experimental-strip-types you added recently, any idea how to fix this? (I don't know the node ecosystem much so)

@Veykril Veykril force-pushed the push-mnpmxxrprymo branch from 7918812 to 46cd1ce Compare April 21, 2025 07:26
@BenjaminBrienen
Copy link
Contributor

@BenjaminBrienen the typescript CI seems to fail now on the --experimental-strip-types you added recently, any idea how to fix this? (I don't know the node ecosystem much so)

It just wants you to run prettier. npm run format will run the script called format from package.json :)

@Veykril
Copy link
Member Author

Veykril commented Apr 21, 2025

Well no, it is formatted. format:check chokes on the warning generated by the experimental flag

@Veykril
Copy link
Member Author

Veykril commented Apr 21, 2025

Oh, nevermind. It only reports the files that have formatting erorrs on them, but not the formatting issues themselves ...

@Veykril Veykril force-pushed the push-mnpmxxrprymo branch from 46cd1ce to 05b374a Compare April 21, 2025 08:38
@Veykril Veykril added this pull request to the merge queue Apr 21, 2025
Merged via the queue into rust-lang:master with commit 61635c7 Apr 21, 2025
15 checks passed
@Veykril Veykril deleted the push-mnpmxxrprymo branch April 21, 2025 12:00
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.

setting rust-analyzer.server.extraEnv.key = null crashes the TS extension
4 participants