Skip to content

Source gen#44

Closed
andreakarasho wants to merge 36 commits intomainfrom
source-gen
Closed

Source gen#44
andreakarasho wants to merge 36 commits intomainfrom
source-gen

Conversation

@andreakarasho
Copy link
Owner

No description provided.

@andreakarasho andreakarasho marked this pull request as ready for review September 19, 2025 14:26
Copilot AI review requested due to automatic review settings September 19, 2025 14:26
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 introduces a new C# source generator for TinyEcs that automatically generates adapter classes for methods decorated with the [TinySystem] attribute. This modernizes the system registration process by providing compile-time code generation instead of manual system creation.

Key changes:

  • Add source generator that creates adapter classes for [TinySystem] methods
  • Refactor system architecture to use new adapter pattern with ITinySystem interface
  • Update samples to demonstrate the new attribute-based system definition

Reviewed Changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/TinyEcs.SourceGenerator/Program.cs Main source generator that analyzes methods with [TinySystem] and generates adapter classes
tools/TinyEcs.SourceGenerator/IndentedStringBuilder.cs String builder utility for generating properly formatted C# code
src/Bevy.cs Major refactoring introducing new system architecture with ITinySystem interface and Stage-based scheduling
samples/MyBattleground/Program.cs Updated to use the new [TinySystem] attribute for system definitions
samples/TinyEcsGame/Program.cs Updated to use the new system architecture and attribute-based systems

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@andreakarasho
Copy link
Owner Author

I dont like it.

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