Skip to content

Feature/copilot passthrough p2 - #82

Merged
berthertogen merged 6 commits into
mainfrom
feature/copilot-passthrough-p2
Apr 18, 2026
Merged

Feature/copilot passthrough p2#82
berthertogen merged 6 commits into
mainfrom
feature/copilot-passthrough-p2

Conversation

@berthertogen

Copy link
Copy Markdown
Owner
  • Update sample apps to use local project references
  • Refactor API sample to reuse a DI-managed Printer
  • Initialize Printer once at app startup (not per request)
  • Fix README script naming/instructions to match repo reality
  • Build + test to confirm no regressions

Closes #80

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the sample applications to reference the local Weasyprint.Wrapped project and refactors the API sample to reuse a DI-managed Printer that is initialized once at startup, while also aligning README instructions with current script names.

Changes:

  • Replaced NuGet package references in sample apps with local ProjectReferences.
  • Refactored ExampleApi to inject a singleton Printer and initialize it once during app startup.
  • Updated README to reference build-on-linux.sh and adjusted sample-running guidance.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/Weasyprint.Wrapped.ExampleApi/Weasyprint.Wrapped.ExampleApi.csproj Switches API sample from package reference to project reference.
src/Weasyprint.Wrapped.ExampleApi/Program.cs Registers Printer as a singleton and initializes it at startup.
src/Weasyprint.Wrapped.ExampleApi/Controllers/PrintController.cs Fixes logger generic type and injects/reuses DI-managed Printer.
src/Weasyprint.Wrapped.Example/Weasyprint.Wrapped.Example.csproj Switches console sample from package reference to project reference.
readme.md Updates Linux build script name and revises sample-running instructions.

Comment thread readme.md Outdated
Comment thread src/Weasyprint.Wrapped.Example/Weasyprint.Wrapped.Example.csproj
berthertogen and others added 3 commits April 18, 2026 22:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…i.csproj

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@berthertogen
berthertogen enabled auto-merge April 18, 2026 20:17
@berthertogen
berthertogen merged commit ac3597c into main Apr 18, 2026
6 checks passed
@berthertogen
berthertogen deleted the feature/copilot-passthrough-p2 branch April 18, 2026 20:25
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.

Good Quick Wins

2 participants