Java updates#5
Merged
gregschunemann merged 2 commits intomainfrom Sep 12, 2025
Merged
Conversation
…tation for clarity for the Angular exercises
Contributor
There was a problem hiding this comment.
Pull Request Overview
Updates documentation and configuration for Java PetClinic and Angular/TypeScript Library projects to clarify technology stacks, improve feature suggestions, and refine setup instructions.
- Switches Java PetClinic from "ConPort workflow" to "Spec-Driven workflow" with updated Java 17/Spring Boot stack
- Expands feature idea lists with more detailed, modern options for both Java and TypeScript projects
- Clarifies Node.js compatibility requirements and removes unused VSCode extensions
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| goals/typescript/2-requirements.md | Updates technology stack info and replaces feature suggestions with more specific, modern options |
| goals/typescript/1-setup.md | Adds Node.js version compatibility clarification and removes Docker recommendation |
| goals/java/2-requirements.md | Changes workflow reference, updates tech stack to Java 17/Spring Boot, and expands feature ideas significantly |
| goals/java/1-setup.md | Removes mention of Microsoft.Learn MCP tool |
| README.md | Clarifies TypeScript support status for Angular vs React |
| .vscode/extensions.json | Removes unused mermaidchart extension |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
gregschunemann
approved these changes
Sep 12, 2025
Collaborator
gregschunemann
left a comment
There was a problem hiding this comment.
Looks great - thanks for doing this.
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.
This pull request updates documentation and configuration for both the Java PetClinic and Angular/TypeScript Library projects to clarify technology stacks, improve feature suggestion lists, and refine setup instructions. It also removes unused extensions from the VSCode configuration.
Documentation and Requirements Updates
goals/java/2-requirements.md) to reflect a switch from the "ConPort workflow" to "Spec-Driven workflow," revised the technology stack to Java 17/Spring Boot, and expanded the list of beginner, intermediate, and advanced feature ideas with more detailed and modern options. [1] [2]goals/typescript/2-requirements.md) to clarify Node.js compatibility, expand beginner and intermediate feature ideas, and introduce advanced migration and mobile app options.Technology Stack and Setup Clarifications
goals/typescript/1-setup.md) for Node.js version compatibility for Angular apps and removed the Docker recommendation.Configuration Cleanup
mermaidchart.vscode-mermaid-chartextension from.vscode/extensions.jsonto streamline the recommended VSCode extensions.Minor Documentation Correction