Import changes from previous project iterations. #1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Imported the primary domain interfaces from previous project iterations. * Resolved issue where ILayer did not condition TDomain to extend from IDomain. * Added a latest build shield to the README.md * Imported the .editorconfig from previous project iterations.
* Imported the primary layer interfaces from previous project iterations.
* Imported the primary connector interfaces from previous project iterations.
* Imported the primary interaction interfaces from previous project iterations.
* Imported the core debugging classes from previous project iterations.
* Major modifications were made to CatalystDebug.cs to extract and unify certain sections of code, including but not limited to:
* CatalystDebugConfiguration
* CatalystDebugOptions
* Imported the profiling project from previous project iterations. * No additional changes were made.
* Imported the collections project from previous project iterations. * No additional changes were made other than bumping the version to beta.2.
* Imported the threading and supporting projects from previous project iterations, including:
* Core/CatalystUI.Threading
* Core/CatalystUI.Attributes
* Tooling/CatalystUI.Analyzers
* Tooling/CatalystUI.CodeFix
* Bumped all versions to beta.2.
* Updated all project authors from "FireController#1847" to "CatalystUI LLC"
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR imports and integrates several core CatalystUI projects from previous iterations, establishing foundational APIs for collections, threading, debugging, attributes, analyzers, and code fixes. The changes include minor adjustments for naming consistency (author field updates) and configuration updates to support the expanded project structure.
Key Changes:
- Addition of 7 new projects across Core and Tooling modules
- Update of CatalystUI.Core dependencies to reference newly added projects
- Extension of CI/CD workflow to support alpha and beta branches
Reviewed Changes
Copilot reviewed 63 out of 64 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Added GitHub Actions workflow status badge |
| CatalystUI/Core/CatalystUI.Core/CatalystUI.Core.csproj | Updated author and added dependencies for analyzers, attributes, collections, and threading |
| CatalystUI/Core/CatalystUI.Collections/*.cs | Added static array-based collection implementations (Stack, Queue, Pool) |
| CatalystUI/Core/CatalystUI.Threading/*.cs | Added thread delegate dispatcher and delegate queue infrastructure |
| CatalystUI/Core/CatalystUI.Attributes/Threading/CachedDelegateAttribute.cs | Added attribute for generating cached delegate references |
| CatalystUI/Core/CatalystUI.Core/Debugging/*.cs | Added debugging infrastructure with configurable logging and context management |
| CatalystUI/Core/CatalystUI.Core/Domains/*.cs | Added domain interfaces for CatalystUI model |
| CatalystUI/Core/CatalystUI.Core/Layers/*.cs | Added layer interfaces for CatalystUI architecture |
| CatalystUI/Core/CatalystUI.Core/Connectors/*.cs | Added connector interfaces linking layers |
| CatalystUI/Core/CatalystUI.Core/Interactions/*.cs | Added interaction and input device interfaces |
| CatalystUI/Tooling/CatalystUI.Analyzers/*.cs | Added Roslyn analyzers for thread delegate dispatcher and cached delegate patterns |
| CatalystUI/Tooling/CatalystUI.CodeFix/*.cs | Added code fix providers for analyzer diagnostics |
| CatalystUI/Tooling/CatalystUI.Profiling/* | Added profiling tool project structure with templates |
| CatalystUI/.editorconfig | Added comprehensive C# formatting and naming conventions |
| .scripts/Setup.ps1 | Updated to include new projects in setup script |
| .github/workflows/dotnet.yml | Extended workflow triggers to alpha and beta branches |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: FireController#1847 <usfirepilot123@gmail.com>
Contributor
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 63 out of 64 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
* Temporarily remove dependency on Catalyst.Debug for the Tooling/CatalystUI.Profiling project, since that has not yet been re-added.
Signed-off-by: FireController#1847 <usfirepilot123@gmail.com>
FireController1847
added
ACT: Review
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📑 Summary
Beginning process for importing previous project iterations with minor changes made across the board for unification.
⚒️ Changes
Caution
Accurately describing the changes made in this PR will greatly improve the speed at which we can review your PR. Semantic Versioning can be helpful in determining the scope of your change, and will help in determining the appropriate version number, merge location, and timing of the integration of your change.
➕ Additions
Added all of the following projects from previous project iterations:
✖️ Modifications
All changes are noted in the individual commits. Most changes were minor. The most extensive changes made were to the
CatalystDebugclass inCatalyst.Debugging.➖ Removals
💬 Review
Important
Accurately listing key items for review will help us review your PR more quickly. Please include large changes, breaking changes, and any other changes that may require special attention.
Reviewers will check off items in the list as they are reviewed, in addition to the normal GitHub review process. They may request changes if any items are not adequately addressed or add additional items as necessary.