feat(settings): add HVE Core extension to workspace and devcontainer recommendations#226
feat(settings): add HVE Core extension to workspace and devcontainer recommendations#226WilliamBerryiii wants to merge 4 commits intomainfrom
Conversation
…recommendations - update extension ID from hve-core to hve-core-all in .vscode/extensions.json - add ise-hve-essentials.hve-core-all to .devcontainer/devcontainer.json extensions - add Built with HVE Core attribution to README.md acknowledgments 🧩 - Generated by Copilot
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #226 +/- ##
=======================================
Coverage 50.46% 50.46%
=======================================
Files 267 267
Lines 18098 18098
Branches 1855 1903 +48
=======================================
Hits 9134 9134
Misses 8674 8674
Partials 290 290
*This pull request uses carry forward flags. Click here to find out more. 🚀 New features to boost your workflow:
|
| "GitHub.copilot@prerelease", | ||
| "GitHub.copilot-chat@prerelease" | ||
| "GitHub.copilot-chat@prerelease", | ||
| "ise-hve-essentials.hve-core-all" |
There was a problem hiding this comment.
We dont want hve-core-all, just hve-core please, unless we want others with it.
There was a problem hiding this comment.
I made this ALL under the theory that we'd want to be able to support teams with being able to run the full project planning through platform building through to platform operations lifecycle in the context of the accelerator itself. There's also the work on the accelerator (which would benefit from the full library), we also have DS components, Security concerns, RAI, programming/scripting, etc. ... by the time we support all the persona using/contributing to the accelerator ... we need all of HVE Core.
| "ms-python.vscode-pylance", | ||
| "ms-python.python", | ||
| "ise-hve-essentials.hve-core", | ||
| "ise-hve-essentials.hve-core-all", |
There was a problem hiding this comment.
Same here, not hve-core-all just hve-core
katriendg
left a comment
katriendg
left a comment
There was a problem hiding this comment.
I believe this is the collection we need, also prevents from having duplicate items for some of the agents which are across more than one collection.
I'll wait for @agreaves-ms to chime back in ... but I agree. |
5 participants
Pull Request
Description
Add HVE Core extension pack (
ise-hve-essentials.hve-core-all) to workspace and devcontainer extension recommendations, ensuring consistent tooling for all contributors. Attribute HVE Core in README acknowledgments.Closes #132
Type of Change
Component(s) Affected
deploy/000-prerequisites- Azure subscription setupdeploy/001-iac- Terraform infrastructuredeploy/002-setup- OSMO control plane / Helmdeploy/004-workflow- Training workflowssrc/training- Python training scriptsdocs/- DocumentationTesting Performed
planreviewed (no unexpected changes)applytested in dev environmentsmoke_test_azure.py)Additional validation:
npm run spell-check— 419 files, 0 issuesnpm run lint:md— 93 files, 0 errors.vscode/extensions.jsonand.devcontainer/devcontainer.jsonDocumentation Impact
Bug Fix Checklist
N/A — this is a feature PR, not a bug fix.
Checklist