Consolidate workspace configurations by removing old DotNet and Python workspace files and adding a unified ProjectTemplate workspace file - #210
Merged
Conversation
…n workspace files and adding a unified ProjectTemplate workspace file Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR consolidates VS Code workspace setup by removing the separate language-specific workspace file(s) and extending the unified ProjectTemplate.code-workspace to cover both .NET and Python development.
Changes:
- Removed
Python.code-workspaceto rely on a single workspace configuration. - Updated
ProjectTemplate.code-workspaceto include Python editor settings and extension recommendations.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| Python.code-workspace | Deleted the standalone Python workspace configuration as part of consolidation. |
| ProjectTemplate.code-workspace | Added Python-specific settings/extensions and adjusted workspace content to serve as the unified configuration. |
Comments suppressed due to low confidence (2)
ProjectTemplate.code-workspace:6
- The workspace file's "folders" block uses tab indentation, but the rest of the file uses spaces and .editorconfig defaults to indent_style=space. This makes formatting inconsistent and can cause noisy diffs when auto-formatting.
ProjectTemplate.code-workspace:54 - "purpleair" was removed from cSpell.words, but the term still appears in repo docs (e.g., README.md and AGENTS.md). Keeping it in the workspace dictionary avoids persistent spell-check false positives for contributors.
ptr727
added a commit
that referenced
this pull request
Jun 26, 2026
- Reword "never copied into derived repos" (README + AGENTS.md): the JSON is maintained in the template as the source of truth and reconciled against a repo's live config; it is not carried/re-synced as a per-repo copy. Avoids the inaccurate absolute (a template-instantiated repo does receive the files). - Mirror the two new general extensions (actionlint, shellcheck) into both devcontainers and fix the stale `*.code-workspace` reference (the per-language workspaces were consolidated into ProjectTemplate.code-workspace in #210). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ptr727
added a commit
that referenced
this pull request
Jul 4, 2026
Promote the accumulated `develop` work to `main`. Twelve changes since the last promotion: - Add `WORKFLOW.md`: workflow style, architecture, behavioral contract, test methodology (#223) - Use NBGV `IGNORE_GITHUB_REF` instead of the ineffective `GITHUB_REF` override (#222) - Skip validate-release on smoke builds (#220) - Gate asset delete on the release create/refresh condition (#218) - Ship branch rulesets as versioned JSON in the re-sync / drift loop (#212) - Clean up transfer artifacts surgically at consumption, not blanket-delete (#216) - Version each publish leg against its own branch; validate at entry (#215) - Consolidate workspace configurations into a unified ProjectTemplate workspace (#210) - Template convergence barrier: absorb pins, generic release + docker-readme, carry-whole-file (#207) - Key merge-bot concurrency on PR number, not `github.ref` (#206) - Codify orchestrated re-sync personas and full-replacement rule (#205) - Lead action pins; affirm pattern-based artifact handoff (#204) 🤖 Generated with [Claude Code](https://claude.com/claude-code)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Pieter Viljoen ptr727@users.noreply.github.com