Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

Implement clone on ReportBuilder#16

Open
cswinter wants to merge 2 commits intozesterer:mainfrom
cswinter:patch-1
Open

Implement clone on ReportBuilder#16
cswinter wants to merge 2 commits intozesterer:mainfrom
cswinter:patch-1

Conversation

@cswinter
Copy link
Copy Markdown

Fixes #15

@zesterer
Copy link
Copy Markdown
Owner

Thanks!

@zesterer
Copy link
Copy Markdown
Owner

Looks like there's a bit more to do on this PR: adding a few Clone constraints in derives. It's likely going to be a breaking change.

@mxthevs
Copy link
Copy Markdown

mxthevs commented Jun 12, 2022

What is the status of this PR? I would love to have a clone method both on ReportBuilder and Report :)

@zesterer
Copy link
Copy Markdown
Owner

It looks like it just needs someone to pick it up again.

@TornaxO7
Copy link
Copy Markdown

TornaxO7 commented Jun 14, 2022

I looked a little bit, if I can pick this PR, but when I saw this, I was a little unsure what the best approach would be.

@zesterer
Copy link
Copy Markdown
Owner

The comment in that file only really relates to the logic for actually generating a diagnostic. Implementing Clone for ReportBuilder is just a case of writing a manual Clone impl with the correct bounds.

@zesterer zesterer force-pushed the main branch 2 times, most recently from 2f36bc2 to 4b3807c Compare December 13, 2025 21:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement clone on ReportBuilder

4 participants