Skip to content

docs: add post-change obligations and Sonar quality guidance to AGENTS.md#93

Merged
phoneuooyd merged 1 commit intomainfrom
codex/update-agents.md-with-responsibilities-and-guidelines
Mar 6, 2026
Merged

docs: add post-change obligations and Sonar quality guidance to AGENTS.md#93
phoneuooyd merged 1 commit intomainfrom
codex/update-agents.md-with-responsibilities-and-guidelines

Conversation

@phoneuooyd
Copy link
Copy Markdown
Owner

Motivation

  • Zapewnić, że po każdej zmianie w kodzie nastąpi przegląd i ewentualna aktualizacja kluczowej dokumentacji projektu.
  • Wprowadzić wymóg stosowania zasad jakości .NET zgodnych z SonarLint/SonarQube jako minimalny standard jakości kodu.

Description

  • Dodano sekcję ## Obowiązki po zmianach w kodzie z regułą przeglądu/aktualizacji plików FoodbookApp.App/DOCS/DOCUMENTATION-GUIDE.md, FoodbookApp.App/DOCS/ARCHITECTURE.md i FoodbookApp.App/DOCS/PROJECT-FILES.md po każdej zmianie w kodzie.
  • Dodano sekcję ## Jakość kodu (Sonar) z wymaganiem zgodności z SonarLint/SonarQube oraz listą minimalnych oczekiwań (usuwanie code smell, unikanie duplikacji, poprawna obsługa wyjątków, brak martwego kodu, zgodność z nullable, dobre praktyki async/await).
  • Dodano ### Checklista przed zamknięciem zadania obejmującą aktualizację kodu, dokumentacji oraz spełnienie wymagań jakościowych Sonar.
  • Zaktualizowano metadane dokumentu (Ostatnia aktualizacja i Wersja).

Testing

  • Uruchomione polecenia selekcyjne w repozytorium to m.in. rg --files -g 'AGENTS.md' i cat FoodbookApp.App/DOCS/AGENTS.md w celu weryfikacji pliku, które zakończyły się sukcesem.
  • Wykonano git diff, git add i git commit aby zatwierdzić zmiany, które zakończyły się sukcesem (commit utworzony).
  • Sprawdzono historię za pomocą git log -1 --oneline oraz stan repozytorium git status --short, które potwierdziły prawidłowe zastosowanie zmian; wszystkie powyższe automatyczne kontrole zakończyły się pomyślnie.

Codex Task

Copy link
Copy Markdown
Owner Author

@phoneuooyd phoneuooyd left a comment

Choose a reason for hiding this comment

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

ok

@phoneuooyd phoneuooyd merged commit 992d78e into main Mar 6, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant