Skip to content

feat: fix devcontainer for rust analyzer#1289

Merged
preinlein merged 1 commit intomainfrom
paul.reinlein/fix-devcontainer-for-rust-analyzer
Mar 21, 2025
Merged

feat: fix devcontainer for rust analyzer#1289
preinlein merged 1 commit intomainfrom
paul.reinlein/fix-devcontainer-for-rust-analyzer

Conversation

@preinlein
Copy link
Copy Markdown
Contributor

@preinlein preinlein commented Mar 20, 2025

What does this PR do?

This addresses some issues with the devcontainer and working in VsCode/Cursor: the rust analyzer was running into some errors around openssl/pkg-config.

With these changes, you should be able to launch VsCode/Cursor in a devcontainer and get appropriate warnings/errors in your IDE as rust-analyzer now runs.

Motivation

I wasn't seeing errors in my IDE that should have been obvious.

Related issues

N/A

Additional Notes

N/A

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@preinlein preinlein marked this pull request as ready for review March 20, 2025 21:30
@preinlein preinlein requested a review from a team as a code owner March 20, 2025 21:30
@preinlein preinlein force-pushed the paul.reinlein/fix-devcontainer-for-rust-analyzer branch from c1f8245 to c8f2ba7 Compare March 20, 2025 21:31
},
"ghcr.io/devcontainers/features/rust:1": {
"version": "1.70",
"version": "1.77",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If possible, this version should match the version in

channel = "1.85.0"
.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure where I grabbed 1.77 from...updated it

@preinlein preinlein force-pushed the paul.reinlein/fix-devcontainer-for-rust-analyzer branch from c8f2ba7 to 4ef19a3 Compare March 20, 2025 21:39
@preinlein preinlein merged commit 018a806 into main Mar 21, 2025
21 checks passed
@preinlein preinlein deleted the paul.reinlein/fix-devcontainer-for-rust-analyzer branch March 21, 2025 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants