Skip to content

Commit fe6e0d1

Browse files
authored
Merge pull request #261 from nanotaboada/chore/257-remove-codacy-integration
chore(ci): remove Codacy integration (#257)
2 parents 502e4aa + 26667e2 commit fe6e0d1

File tree

6 files changed

+20
-45
lines changed

6 files changed

+20
-45
lines changed

.codacy.yml

Lines changed: 0 additions & 16 deletions
This file was deleted.

.dockerignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
.mvn
33
.vscode
44
target
5-
.codacy.yml
65
.gitignore
76
azure-pipelines.yml
87
CODE_OF_CONDUCT.md

.github/copilot-instructions.md

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,30 @@
11
# GitHub Copilot Instructions
22

3-
> **Token Efficiency Note**: This is a minimal pointer file (~500 tokens, auto-loaded by Copilot).
4-
> For complete operational details, reference: `#file:AGENTS.md` (~2,500 tokens, loaded on-demand)
3+
> **Token Efficiency Note**: This is a minimal pointer file (~500 tokens, auto-loaded by Copilot).
4+
> For complete operational details, reference: `#file:AGENTS.md` (~2,500 tokens, loaded on-demand)
55
> For specialized knowledge, use: `#file:SKILLS/<skill-name>/SKILL.md` (loaded on-demand when needed)
66
7-
## 🎯 Quick Context
7+
## Quick Context
88

9-
**Project**: Spring Boot REST API demonstrating modern Java patterns
10-
**Stack**: Java 25 (LTS) • Spring Boot 4 • JPA/Hibernate • SQLite • Maven • Docker
11-
**Pattern**: Controller → Service → Repository → JPA (layered architecture)
9+
**Project**: Spring Boot REST API demonstrating modern Java patterns
10+
**Stack**: Java 25 (LTS) • Spring Boot 4 • JPA/Hibernate • SQLite • Maven • Docker
11+
**Pattern**: Controller → Service → Repository → JPA (layered architecture)
1212
**Philosophy**: Learning-focused PoC emphasizing Spring Boot best practices
1313

14-
## 📐 Core Conventions
14+
## Core Conventions
1515

1616
- **Naming**: camelCase (methods/variables), PascalCase (classes)
1717
- **Annotations**: Use Spring stereotypes (@RestController, @Service, @Repository)
1818
- **Lombok**: Reduce boilerplate (@Data, @Builder, @AllArgsConstructor)
1919
- **Dependency Injection**: Constructor injection (Lombok @RequiredArgsConstructor)
2020
- **Testing**: JUnit 5 + AssertJ for fluent assertions
2121
- **Build**: Use `./mvnw` wrapper, NOT system Maven
22+
- **Commit Messages**: Follow Conventional Commits with issue number suffix
23+
- Format: `type(scope): description (#issue)` (max 80 chars)
24+
- Example: `docs: optimize AI agent instructions for token efficiency (#259)`
25+
- Types: `feat`, `fix`, `chore`, `docs`, `test`, `refactor`
2226

23-
## 🏗️ Architecture at a Glance
27+
## Architecture at a Glance
2428

2529
```
2630
Controller → Service → Repository → JPA → Database
@@ -34,7 +38,7 @@ Validation Cache Query Methods
3438
- **DTOs**: ModelMapper for entity ↔ DTO transformations
3539
- **Cache**: Spring Cache abstraction (1-hour TTL)
3640

37-
## Copilot Should
41+
## Copilot Should
3842

3943
- Generate idiomatic Spring Boot code with proper annotations
4044
- Use JPA repository patterns (derived queries, @Query)
@@ -44,7 +48,7 @@ Validation Cache Query Methods
4448
- Use ModelMapper for DTO transformations
4549
- Implement proper exception handling with @ControllerAdvice
4650

47-
## 🚫 Copilot Should Avoid
51+
## Copilot Should Avoid
4852

4953
- Field injection (use constructor injection)
5054
- Using `new` for services (breaks DI)
@@ -53,7 +57,7 @@ Validation Cache Query Methods
5357
- System.out.println (use SLF4J logging)
5458
- Hardcoded configuration (use @Value or application.yml)
5559

56-
## Quick Commands
60+
## Quick Commands
5761

5862
```bash
5963
# Run with hot reload
@@ -69,12 +73,12 @@ docker compose up
6973
# Actuator: http://localhost:9001/actuator/health
7074
```
7175

72-
## 📚 Need More Detail?
76+
## Need More Detail?
7377

74-
**For operational procedures**: Load `#file:AGENTS.md`
75-
**For Docker expertise**: *(Planned)* `#file:SKILLS/docker-containerization/SKILL.md`
78+
**For operational procedures**: Load `#file:AGENTS.md`
79+
**For Docker expertise**: *(Planned)* `#file:SKILLS/docker-containerization/SKILL.md`
7680
**For testing patterns**: *(Planned)* `#file:SKILLS/testing-patterns/SKILL.md`
7781

7882
---
7983

80-
💡 **Why this structure?** Copilot auto-loads this file on every chat (~500 tokens). Loading `AGENTS.md` or `SKILLS/` explicitly gives you deep context only when needed, saving 80% of your token budget!
84+
**Why this structure?** Copilot auto-loads this file on every chat (~500 tokens). Loading `AGENTS.md` or `SKILLS/` explicitly gives you deep context only when needed, saving 80% of your token budget!

.github/workflows/maven.yml

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,6 @@ jobs:
4242
runs-on: ubuntu-latest
4343
permissions:
4444
contents: read
45-
strategy:
46-
matrix:
47-
service: [codecov, codacy]
4845
steps:
4946
- name: Checkout repository
5047
uses: actions/checkout@v6.0.2
@@ -55,19 +52,11 @@ jobs:
5552
name: jacoco.xml
5653

5754
- name: Upload JaCoCo coverage report to Codecov
58-
if: matrix.service == 'codecov'
5955
uses: codecov/codecov-action@v5.5.2
6056
with:
6157
token: ${{ secrets.CODECOV_TOKEN }}
6258
files: jacoco.xml
6359

64-
- name: Upload JaCoCo coverage report to Codacy
65-
if: matrix.service == 'codacy'
66-
uses: codacy/codacy-coverage-reporter-action@v1.3.0
67-
with:
68-
project-token: ${{ secrets.CODACY_PROJECT_TOKEN }}
69-
force-coverage-parser: jacoco -r jacoco.xml
70-
7160
container:
7261
needs: coverage
7362
runs-on: ubuntu-latest

AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ curl http://localhost:8080/actuator/health
160160
2. **Lint**: Commit message validation (commitlint)
161161
3. **Build**: `./mvnw clean install` (compile + test + package)
162162
4. **Test**: Tests already run during install, coverage reports generated
163-
5. **Coverage**: JaCoCo report upload to Codecov and Codacy
163+
5. **Coverage**: JaCoCo report upload to Codecov
164164

165165
**Local validation** (run this before pushing):
166166

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
[![Java CI with Maven](https://github.com/nanotaboada/java.samples.spring.boot/actions/workflows/maven.yml/badge.svg)](https://github.com/nanotaboada/java.samples.spring.boot/actions/workflows/maven.yml)
44
[![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=nanotaboada_java.samples.spring.boot&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=nanotaboada_java.samples.spring.boot)
5-
[![Codacy Badge](https://app.codacy.com/project/badge/Grade/ea6918db332642089ef420822267931b)](https://app.codacy.com/gh/nanotaboada/java.samples.spring.boot/dashboard?utm_source=gh&utm_medium=referral&utm_content=&utm_campaign=Badge_grade)
65
[![codecov](https://codecov.io/gh/nanotaboada/java.samples.spring.boot/branch/master/graph/badge.svg?token=D3FMNG0WOI)](https://codecov.io/gh/nanotaboada/java.samples.spring.boot)
76
[![CodeFactor](https://www.codefactor.io/repository/github/nanotaboada/java.samples.spring.boot/badge)](https://www.codefactor.io/repository/github/nanotaboada/java.samples.spring.boot)
87
[![License: MIT](https://img.shields.io/badge/License-MIT-white.svg)](https://opensource.org/licenses/MIT)

0 commit comments

Comments
 (0)