Skip to content

Feature/simondarksidej/itemtemplates#37

Merged
SimonDarksideJ merged 3 commits intomainfrom
feature/simondarksidej/itemtemplates
Apr 20, 2026
Merged

Feature/simondarksidej/itemtemplates#37
SimonDarksideJ merged 3 commits intomainfrom
feature/simondarksidej/itemtemplates

Conversation

@SimonDarksideJ
Copy link
Copy Markdown
Contributor

Summary

For the new Content Builder solution, added two new item templates:

  • SpriteFont template to replace the MGCB "SpriteFont" template.
  • Content Pipeline Extension, for creating an extension within the Builder project.

Copilot AI review requested due to automatic review settings January 19, 2026 14:50
Copy link
Copy Markdown
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 two new item templates for the MonoGame Content Builder solution: a SpriteFont template and a Content Pipeline Extension template. The README.md is significantly restructured and expanded to document both existing project templates and the new item templates with usage examples.

Changes:

  • Added SpriteFont item template (mgsf) with configurable font inclusion
  • Added Content Pipeline Extension item template (mgpipelineitem) for creating importers and processors
  • Restructured and expanded README.md with comprehensive template documentation including usage guidelines and examples

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
README.md Restructured documentation with sections for project templates, item templates, and naming rules
CSharp/content/MonoGame.ItemTemplate.SpriteFont.CSharp/MGSpriteFont.spritefont SpriteFont XML definition with multi-language character support (Latin, Hiragana, Katakana, Kanji, Cyrillic)
CSharp/content/MonoGame.ItemTemplate.SpriteFont.CSharp/.template.config/template.json Template configuration for SpriteFont with optional font file inclusion
CSharp/content/MonoGame.ItemTemplate.Pipeline.Extension.CSharp/MG-Processor.cs Template processor class for content pipeline extensions
CSharp/content/MonoGame.ItemTemplate.Pipeline.Extension.CSharp/MG-Importer.cs Template importer class for content pipeline extensions
CSharp/content/MonoGame.ItemTemplate.Pipeline.Extension.CSharp/.template.config/template.json Template configuration for pipeline extension items

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md
Comment thread README.md
@tomspilman
Copy link
Copy Markdown
Member

How do we disable this useless CoPilot noise?

@SimonDarksideJ
Copy link
Copy Markdown
Contributor Author

How do we disable this useless CoPilot noise?

I already disabled it on my account as Dean suggested. I'll try adding the "no no we don't care" ruleset to this repo too

Any comments on the PR @tomspilman ?

@dellis1972
Copy link
Copy Markdown
Contributor

@SimonDarksideJ do we have the rights to ship that font?

@SimonDarksideJ
Copy link
Copy Markdown
Contributor Author

@SimonDarksideJ do we have the rights to ship that font?

I believe so, it is the font we switched to for the samples, as a Google open source redistribute font

@SimonDarksideJ SimonDarksideJ self-assigned this Apr 6, 2026
@SimonDarksideJ SimonDarksideJ merged commit 6dd2e30 into main Apr 20, 2026
1 check passed
@SimonDarksideJ SimonDarksideJ deleted the feature/simondarksidej/itemtemplates branch April 20, 2026 12:57
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.

4 participants