Skip to content

Fix reportgenerator#2645

Merged
t-b merged 2 commits intomainfrom
bugfix/2645-reportgenerator
Mar 3, 2026
Merged

Fix reportgenerator#2645
t-b merged 2 commits intomainfrom
bugfix/2645-reportgenerator

Conversation

@t-b
Copy link
Collaborator

@t-b t-b commented Mar 3, 2026

Our CI job has this

2026-03-03T14:44:37: Report type 'PngChart' is no longer supported. Please use 'SvgChart' instead.

[1]: https://github.com/AllenInstitute/MIES/actions/runs/22622405584/job/65570244502
@t-b t-b self-assigned this Mar 3, 2026
Copilot AI review requested due to automatic review settings March 3, 2026 19:34
Copy link

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

Updates the local ReportGenerator invocation to address upstream ReportGenerator behavior changes (see linked ReportGenerator issue #776) by switching the generated chart format.

Changes:

  • Replace PngChart with SvgChart in the ReportGenerator -reporttypes list.

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

@t-b t-b marked this pull request as ready for review March 3, 2026 20:23
Copilot AI review requested due to automatic review settings March 3, 2026 20:23
@t-b t-b merged commit d76f9bc into main Mar 3, 2026
4 of 8 checks passed
@t-b t-b deleted the bugfix/2645-reportgenerator branch March 3, 2026 20:24
Copy link

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


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

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.

2 participants