Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add agent instructions - #390

Merged
cleverchuk merged 2 commits into
mainfrom
cc/NH-125033
Dec 8, 2025
Merged

cleverchuk merged 2 commits into
mainfrom
cc/NH-125033

Conversation

@cleverchuk

Copy link
Copy Markdown
Contributor

No description provided.

@cleverchuk
cleverchuk requested a review from a team as a code owner December 5, 2025 17:22
@cleverchuk
cleverchuk requested a review from Copilot December 5, 2025 17:22

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

This PR adds comprehensive agent instructions and documentation files to guide GitHub Copilot and developers working on the SolarWinds APM Java project. The changes establish clear standards for code generation, testing, and build configuration.

Key Changes:

  • Added detailed GitHub Copilot instructions covering project architecture, Java 8 compatibility requirements, and code quality standards
  • Created file-type-specific instruction files for Java, testing, and Gradle configurations
  • Added AGENTS.md developer guide with testing and PR workflow instructions

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/copilot-instructions.md Comprehensive Copilot configuration defining technology stack, architecture, Java 8 compatibility rules, and documentation standards
.github/copilot/instructions/java.md Java-specific coding standards including version compatibility, logging patterns, and code style guidelines
.github/copilot/instructions/test.md Testing conventions for naming, mocking, parameterized tests, and test organization
.github/copilot/instructions/gradle.md Gradle build file standards covering dependency management, Java version configuration, and convention plugins
AGENTS.md Development guide with style guidelines, testing requirements, and PR workflow instructions
Comments suppressed due to low confidence (1)

.github/copilot/instructions/java.md:1

  • The spacing rules are ambiguous and could be misinterpreted. Consider clarifying: (1) what 'at class' means in line 18 (between fields? between field groups?), and (2) whether line 19's 'every 3 statements' means after each group of 3 or between individual statements. Provide concrete code examples to illustrate these rules.
# Java File Instructions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AGENTS.md Outdated

@tammy-baylis-swi tammy-baylis-swi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice!

Co-authored-by: Copilot <[email protected]>
@cleverchuk
cleverchuk merged commit a987136 into main Dec 8, 2025
35 of 39 checks passed
@cleverchuk
cleverchuk deleted the cc/NH-125033 branch December 8, 2025 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants