Skip to content

edburns/java-add-spotless-to-java-coding-skill#1723

Merged
edburns merged 2 commits into
mainfrom
edburns/java-add-spotless-to-java-coding-skill
Jun 18, 2026
Merged

edburns/java-add-spotless-to-java-coding-skill#1723
edburns merged 2 commits into
mainfrom
edburns/java-add-spotless-to-java-coding-skill

Conversation

@edburns

@edburns edburns commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

modified: .github/skills/java-coding-skill/SKILL.md

modified:   .github/skills/java-coding-skill/SKILL.md

- While working on #1721, I discovered and hereby fix this important omission.

Signed-off-by: Ed Burns <edburns@microsoft.com>
Copilot AI review requested due to automatic review settings June 18, 2026 19:17
@edburns edburns requested a review from a team as a code owner June 18, 2026 19:17

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

Updates the Java coding skill documentation to add a missing contributor workflow note about Spotless formatting, aiming to prevent CI failures caused by unformatted Java changes.

Changes:

  • Documented that Java CI enforces mvn spotless:check and contributors should run Spotless before committing.
Show a summary per file
File Description
.github/skills/java-coding-skill/SKILL.md Adds a Best Practices entry about running Spotless to avoid Java CI failures.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread .github/skills/java-coding-skill/SKILL.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@edburns edburns merged commit b9c4a89 into main Jun 18, 2026
10 checks passed
@edburns edburns deleted the edburns/java-add-spotless-to-java-coding-skill branch June 18, 2026 19:33
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.

2 participants