Skip to content

chore: update architecture agent#7382

Merged
JanDeDobbeleer merged 3 commits intomainfrom
next
Mar 12, 2026
Merged

chore: update architecture agent#7382
JanDeDobbeleer merged 3 commits intomainfrom
next

Conversation

@JanDeDobbeleer
Copy link
Copy Markdown
Owner

@JanDeDobbeleer JanDeDobbeleer commented Mar 11, 2026

Prerequisites

  • I have read and understood the contributing guide.
  • The commit message follows the conventional commits guidelines.
  • Tests for the changes have been added (for bug fixes / features).
  • Docs have been added/updated (for bug fixes / features).

Description

Copilot AI review requested due to automatic review settings March 11, 2026 19:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 Architecture and Design agent guidance to incorporate additional design principles and expand the review checklist, helping reviewers apply consistent architectural heuristics across the codebase.

Changes:

  • Add explicit references to Object Calisthenics and Clean Code as guiding principles.
  • Expand the code review checklist with additional design-focused checks (naming, Law of Demeter, primitive obsession, SRP, DRY).

dependabot bot and others added 2 commits March 12, 2026 20:27
Bumps the all group with 1 update: [actions/download-artifact](https://github.com/actions/download-artifact).


Updates `actions/download-artifact` from 8.0.0 to 8.0.1
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@70fc10c...3e5f45b)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: 8.0.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: all
...

Signed-off-by: dependabot[bot] <support@github.com>
@JanDeDobbeleer JanDeDobbeleer enabled auto-merge (rebase) March 12, 2026 19:28
@JanDeDobbeleer JanDeDobbeleer merged commit fce8753 into main Mar 12, 2026
14 checks passed
@JanDeDobbeleer JanDeDobbeleer deleted the next branch March 12, 2026 19:31
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