Skip to content

Add high-level architecture doc#812

Merged
cpuguy83 merged 1 commit intoproject-dalec:mainfrom
cpuguy83:architecture
Oct 28, 2025
Merged

Add high-level architecture doc#812
cpuguy83 merged 1 commit intoproject-dalec:mainfrom
cpuguy83:architecture

Conversation

@cpuguy83
Copy link
Collaborator

@cpuguy83 cpuguy83 commented Oct 23, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 23, 2025 00:32
Copy link
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

This PR adds a comprehensive architecture document (ARCHITECTURE.md) that explains Dalec's high-level design, component structure, and extensibility model. The document serves as a guide for maintainers and contributors to understand how the BuildKit frontend, target plugins, spec parsing, and build orchestration fit together.

Key Changes

  • Introduces an architecture overview covering goals, system components, and data flow
  • Documents the plugin system, target implementations, and packaging workflows
  • Provides guidance on extending Dalec with new targets, sources, and integrations

Copy link
Member

@sozercan sozercan Oct 24, 2025

Choose a reason for hiding this comment

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

can you add this to the website docs

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

@cpuguy83 cpuguy83 force-pushed the architecture branch 3 times, most recently from 5b525e8 to ff518ee Compare October 27, 2025 23:52
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
@cpuguy83 cpuguy83 merged commit 38f926c into project-dalec:main Oct 28, 2025
7 of 8 checks passed
@cpuguy83 cpuguy83 deleted the architecture branch October 28, 2025 22:04
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.

3 participants