Skip to content

Conversation

@kaifcoder
Copy link
Owner

  • Deleted .vscode/settings.json that was forcing git signing settings
  • .vscode/ is already in .gitignore to prevent future additions
  • This prevents the repository from overriding individual developer VS Code preferences

Summary

Explain the change in 1–3 sentences. Reference any related issues (e.g. Closes #123).

Type of Change

  • Feature
  • Bug fix
  • Chore / Refactor
  • Docs
  • Tests
  • CI / Build
  • Other

Motivation / Context

Why is this change needed? What problem does it solve or what capability does it add?

Approach

Briefly describe how you implemented the change. Note any notable design decisions, trade-offs, or alternatives considered.

CLI Impact

If this alters user-facing CLI behavior:

  • Added / changed flags? Describe.
  • Backward compatible? If breaking, explain migration path.
  • Sample invocation before vs after:
# before
# after

Generated Output Impact

List any new / modified scaffold files or structural differences (e.g. new template folder, changed Dockerfile pattern, compose changes, new preset behaviors).

Tests

Describe test coverage:

  • Added new test(s)
  • Updated existing test(s)
  • Manually smoke-tested locally
  • No tests needed (explain why)

If you ran the smoke scaffold locally, paste the command & confirm success:

node bin/index.js demo --services node --no-install --yes

Result: ✅ / ❌

Screenshots / Logs (Optional)

Add any helpful output (chalk-styled CLI messages, error reproduction, etc.).

Docs

  • Updated README.md if needed
  • Updated .github/copilot-instructions.md if internal conventions changed
  • Not applicable

Checklist

  • Code follows existing style (chalk usage, emoji prefixes, exit codes)
  • No accidental large asset additions outside templates/
  • Default ports preserved / conflicts handled
  • New service templates added to: choices array, defaultPorts, Dockerfile switch, compose mapping
  • Git history clean (no stray debug commits)
  • Dependency additions are minimal & justified

Open Questions / Follow-ups

List any TODOs or future enhancements not in this PR.


Thanks for contributing! 🎉

- Deleted .vscode/settings.json that was forcing git signing settings
- .vscode/ is already in .gitignore to prevent future additions
- This prevents the repository from overriding individual developer VS Code preferences
Copilot AI review requested due to automatic review settings October 29, 2025 13:51
@kaifcoder kaifcoder merged commit f93e100 into main Oct 29, 2025
17 checks passed
@kaifcoder kaifcoder deleted the release/new branch October 29, 2025 13:52
Copy link
Contributor

Copilot AI left a 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 is a version bump release from 1.5.0 to 1.6.0, with repository cleanup to properly enforce existing gitignore rules.

  • Version bumped to 1.6.0 in package.json and package-lock.json
  • Removed .vscode/settings.json (should have been gitignored)
  • Added test-workspace/ to .gitignore for test artifact cleanup

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
package.json Version bumped from 1.5.0 to 1.6.0
package-lock.json Lockfile version updated to match package.json
.vscode/settings.json Removed editor-specific settings file (already covered by .gitignore pattern)
.gitignore Added test-workspace/ directory to ignore test artifacts

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