-
Notifications
You must be signed in to change notification settings - Fork 7
fix(cpp): pre-create .amp directory to allow clangd to find it #1025
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
fix(cpp): pre-create .amp directory to allow clangd to find it #1025
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes a tooling issue where clangd ignores configuration arguments when the .amp directory doesn't exist. The change pre-creates the /root/.amp directory during container setup to ensure clangd can properly locate the compile-commands.json file.
Key Changes:
- Pre-create
/root/.ampdirectory in the cpp container's Dockerfile to ensure clangd functionality
✅MegaLinter analysis: Success
See detailed reports in MegaLinter artifacts Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining
|
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
Signed-off-by: Ron <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
|
Pull Request Report (#1025)Static measures
Time related measures
Status check related measures
|
|
🎉 Hooray! The changes in this pull request went live with the release of v6.6.1 🎉 |




🚀 Hey, I have created a Pull Request
Description of changes
This pull request makes a minor adjustment to the
.devcontainer/cpp/Dockerfileby creating a.ampdirectory in the root user's home. This change is necessary for tooling that expects this directory to exist. If it doesn't exist clangd ignores the argument pointing to the directory where compile-commands.json would reside./root/.ampdirectory during container setup.✔️ Checklist