Skip to content

new java and springboot base best practices#22

Merged
aaronpowell merged 18 commits into
github:mainfrom
josephgonzales01:feat/instruction-java-and-springboot
Jul 8, 2025
Merged

new java and springboot base best practices#22
aaronpowell merged 18 commits into
github:mainfrom
josephgonzales01:feat/instruction-java-and-springboot

Conversation

@josephgonzales01

@josephgonzales01 josephgonzales01 commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Title: feat: Expand Java ecosystem prompts and best practices

Description:

This pull request significantly expands the available guidance for Java ecosystem development by introducing a suite of new best practice prompts. These prompts build upon the existing instructions to provide more targeted and actionable advice
for common development tasks, improving consistency and code quality for developers using the agent.

Key Additions:

  • instructions/springboot.instructions.md: Introduces foundational guidelines for all Spring Boot development.
  • instructions/java.instructions.md: Adds Java-specific coding standards and review checks.
  • prompts/java-docs.prompt.md: Establishes best practices for writing comprehensive and standardized Javadoc comments.
  • prompts/java-junit.prompt.md: Outlines modern best practices for writing effective unit tests with JUnit 5, covering test structure, assertions, and parameterized tests.
  • prompts/java-springboot.prompt.md: A detailed guide for developing applications with Spring Boot and Java, covering project structure, dependency injection, testing, and security.
  • prompts/kotlin-springboot.prompt.md: An equivalent guide tailored specifically for idiomatic Kotlin development with Spring Boot, highlighting features like data classes, primary constructors, null safety, and coroutine usage.

@aaronpowell aaronpowell requested a review from Copilot July 3, 2025 01:20

This comment was marked as outdated.

@brunoborges

Copy link
Copy Markdown
Contributor

@josephgonzales01 this is a great start!

May I just suggest that you split Java and Spring instructions into their own individual files?

@josephgonzales01

Copy link
Copy Markdown
Contributor Author

@josephgonzales01 this is a great start!

May I just suggest that you split Java and Spring instructions into their own individual files?

suggestion applied

@brunoborges

Copy link
Copy Markdown
Contributor

This looks great. I'll ask it to be merged. Thanks @josephgonzales01 !

@aaronpowell aaronpowell requested a review from Copilot July 3, 2025 23:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds foundational best practices and review instructions for Java and Spring Boot projects.

  • Introduces springboot.instructions.md with guidelines for Spring Boot development.
  • Introduces java.instructions.md with Java-specific coding standards and checks.

Reviewed Changes

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

File Description
instructions/springboot.instructions.md New Spring Boot development guidelines
instructions/java.instructions.md New Java development guidelines

Comment thread instructions/springboot.instructions.md Outdated
@@ -0,0 +1,58 @@
---
description: 'Guidelines for building Springboot base applications'
applyTo: '**/*.java, **/*.kt,'

Copilot AI Jul 3, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The glob pattern has a trailing comma inside the quotes ('**/.java, **/.kt,') which may cause the applyTo pattern to not match correctly; consider removing the trailing comma or splitting into a list of patterns.

Suggested change
applyTo: '**/*.java, **/*.kt,'
applyTo: '**/*.java, **/*.kt'

Copilot uses AI. Check for mistakes.
Comment thread instructions/springboot.instructions.md Outdated
@@ -0,0 +1,58 @@
---
description: 'Guidelines for building Springboot base applications'

Copilot AI Jul 3, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The term 'Springboot' should be capitalized as 'Spring Boot' to match official naming conventions.

Suggested change
description: 'Guidelines for building Springboot base applications'
description: 'Guidelines for building Spring Boot base applications'

Copilot uses AI. Check for mistakes.
@brunoborges

Copy link
Copy Markdown
Contributor

@josephgonzales01 can you please review the steps documented here?

Also, your PR is out of sync with the latest commit in the main branch.

@josephgonzales01

josephgonzales01 commented Jul 6, 2025

Copy link
Copy Markdown
Contributor Author

Hi Maintainer, I’ve addressed all comments and completed the remaining tasks. The PR is now ready for final review

Comment thread README.md Outdated
@aaronpowell aaronpowell merged commit 1ba5cc7 into github:main Jul 8, 2025
2 checks passed
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.

4 participants