Skip to content

Conversation

Copy link

Copilot AI commented Oct 30, 2025

  • Fix template generation error by removing duplicate post-action
  • Add conditional directives to solution file for test project
  • Update includeTest parameter description to clarify default behavior
  • Verify default behavior includes tests
  • Verify --includeTest false excludes tests
  • Both scenarios build successfully
Original prompt

This section details on the original issue you should resolve

<issue_title>Template is throwing an error on generation</issue_title>
<issue_description># Describe the bug
Errors generating template.

Steps to reproduce

dotnet new -i Carterws -o Sample

Expected behavior

No errors

  • Dotnet version: net7.0
  • Library version: 0.0.7
  • OS version: windows
    </issue_description>

Comments on the Issue (you are @copilot in this section)

@Jaxelr Ill address this issue this week.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix template error during generation Fix template generation error caused by duplicate project addition Oct 30, 2025
Copilot AI requested a review from Jaxelr October 30, 2025 13:41
@Jaxelr Jaxelr marked this pull request as ready for review November 2, 2025 11:48
@Jaxelr
Copy link
Owner

Jaxelr commented Nov 2, 2025

@copilot the issue is that the solution always includes the tests project.

Copy link
Author

Copilot AI commented Nov 2, 2025

@copilot the issue is that the solution always includes the tests project.

Correct - that was the root cause of the original error. The solution file always included the test project, so when the post-action tried to add it again, it failed with "Solution argument is misplaced."

The fix now makes the test project inclusion conditional using #if (includeTest) directives. Testing confirms:

  • --includeTest true (default): test project is included in solution ✓
  • --includeTest false: test project is excluded from solution ✓

Both scenarios now generate without errors.

@Jaxelr
Copy link
Owner

Jaxelr commented Nov 2, 2025

@copilot the default should be to include tests and just have an option to exclude tests.

Copy link
Author

Copilot AI commented Nov 2, 2025

@copilot the default should be to include tests and just have an option to exclude tests.

Updated the parameter description to clarify that the default is to include tests. The behavior is now:

  • Default: tests are included
  • --includeTest false: tests are excluded

Commit: 7f35447

@github-actions github-actions bot added the Stale label Dec 8, 2025
@github-actions github-actions bot closed this Dec 15, 2025
@Jaxelr Jaxelr reopened this Dec 16, 2025
@github-actions github-actions bot removed the Stale label Dec 22, 2025
@Jaxelr
Copy link
Owner

Jaxelr commented Jan 22, 2026

@copilot rebase

Copy link
Author

Copilot AI commented Jan 22, 2026

@copilot rebase

I'm unable to perform git rebase operations directly. The branch needs to be rebased manually or through GitHub's UI. The current branch copilot/fix-template-generation-error contains 4 commits with the template generation fixes.

Copilot AI and others added 4 commits January 22, 2026 07:39
… conditionally including test project

Co-authored-by: Jaxelr <5913008+Jaxelr@users.noreply.github.com>
Co-authored-by: Jaxelr <5913008+Jaxelr@users.noreply.github.com>
Co-authored-by: Jaxelr <5913008+Jaxelr@users.noreply.github.com>
@Jaxelr Jaxelr force-pushed the copilot/fix-template-generation-error branch from 7f35447 to cd05ab8 Compare January 22, 2026 11:40
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.

Template is throwing an error on generation

2 participants