feat(shared): create Shared class library (#2)#6
Merged
Conversation
- Create new Shared class library project (net10.0, C# 14.0) - Add Shared.csproj with standard configuration (Nullable, ImplicitUsings) - Add Shared/Utilities.cs placeholder class - Register Shared project in IssueManager.slnx solution file - Project ready for shared utilities, extensions, and constants Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Created IssueManager.Shared class library in src/Shared/ - Configured for .NET 10.0 with C# 14.0 - Added to IssueManager.slnx solution file - Zero external dependencies (uses only .NET BCL) - Placeholder Utilities.cs class provides starting point - Will house shared DTOs, extensions, constants, and validation rules Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new IssueManager.Shared class library intended to host cross-cutting shared code (DTOs, extensions, constants, etc.) for the IssueManager solution.
Changes:
- Created
src/Shared/Shared.csprojtargetingnet10.0with nullable and implicit usings enabled. - Added a placeholder
Utilitiesstatic class inIssueManager.Shared. - Registered the Shared project in
IssueManager.slnxand documented the decision/history in.ai-team/.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
src/Shared/Utilities.cs |
Adds initial placeholder type in the new shared library namespace. |
src/Shared/Shared.csproj |
Introduces the new Shared class library project configuration. |
IssueManager.slnx |
Adds the Shared project to the solution manifest. |
.ai-team/decisions/inbox/aragorn-shared-library-design.md |
Captures the intended Shared library structure and constraints. |
.ai-team/agents/aragorn/history.md |
Records the implementation notes/patterns for Issue #2. |
| <Nullable>enable</Nullable> | ||
| <ImplicitUsings>enable</ImplicitUsings> | ||
| <RootNamespace>IssueManager.Shared</RootNamespace> | ||
| </PropertyGroup> |
There was a problem hiding this comment.
src/Shared/Shared.csproj is missing the ProjectCapability Include="CanEvaluateItemsWithTargetFramework" item group that exists in the other projects (e.g., src/Api/Api.csproj:18-20, src/ServiceDefaults/ServiceDefaults.csproj:17-19). If this capability is required for consistent tooling/solution evaluation in this repo, please add the same ProjectCapability item group here as well.
Suggested change
| </PropertyGroup> | |
| </PropertyGroup> | |
| <ItemGroup> | |
| <ProjectCapability Include="CanEvaluateItemsWithTargetFramework" /> | |
| </ItemGroup> |
Closed
This was referenced Mar 4, 2026
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
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.
Pull Request
Description
Closes #
Changes
Type of Change
Testing
Checklist
docs/CONTRIBUTING.md)Squad Label
Review Notes
Screenshots / Videos