Conversation
* Enhance module manifest with additional metadata including tags, license, project URL, icon, and release notes * Add validation for category names in StackedBarChart and implement tests for chart functions * Update Pester workflow to use actions/checkout@v6 and setup .NET 8.0, streamline build process for MacOS, Linux, and Windows * Enhance Pester workflow to include conditional checks for OS-specific build and copy steps * Add Windows PowerShell build and copy steps to Pester workflow * Enhance Pester and Release workflows for MacOS ARM64 support and update architecture handling in PowerShell module * Remove unnecessary module dependencies from Invoke-Tests script and clean up Pester workflow * Remove conditional code coverage for Windows PowerShell in Pester workflow * Enhance module import statements for macOS ARM and x86 architectures with verbose and debug output * Refactor workflows and issue templates for consistency and clarity; enhance Pester test output with verbose and debug options * Update Pester workflow to list DLL files from macOS ARM directory * Fix path for listing DLL files in macOS ARM directory in Pester workflow * Fix path for listing DLL files in macOS ARM directory in Pester workflow * Update paths for copying libraries and listing DLL files in Pester workflow * Update Pester workflow to use PowerShell commands for copying libraries across platforms * Update Pester workflow to improve library copy commands and list DLL files for macOS ARM * Update Pester workflow to use Unix-style copy command for macOS library * Update Pester and Release workflows to use PowerShell copy commands and adjust library paths for macOS * Fix module import paths for macOS ARM and x64 architectures in AsBuiltReport.Chart.psm1 * Update module version to 0.2.0 and add changelog entry for new features * Update issue templates: enhance bug report and change request forms
Removed commented-out sections related to tweeting and Bluesky posting.
Updated README.md to reflect current module status and requirements.
* Enhance module manifest with additional metadata including tags, license, project URL, icon, and release notes * Add validation for category names in StackedBarChart and implement tests for chart functions * Update Pester workflow to use actions/checkout@v6 and setup .NET 8.0, streamline build process for MacOS, Linux, and Windows * Enhance Pester workflow to include conditional checks for OS-specific build and copy steps * Add Windows PowerShell build and copy steps to Pester workflow * Enhance Pester and Release workflows for MacOS ARM64 support and update architecture handling in PowerShell module * Remove unnecessary module dependencies from Invoke-Tests script and clean up Pester workflow * Remove conditional code coverage for Windows PowerShell in Pester workflow * Enhance module import statements for macOS ARM and x86 architectures with verbose and debug output * Refactor workflows and issue templates for consistency and clarity; enhance Pester test output with verbose and debug options * Update Pester workflow to list DLL files from macOS ARM directory * Fix path for listing DLL files in macOS ARM directory in Pester workflow * Fix path for listing DLL files in macOS ARM directory in Pester workflow * Update paths for copying libraries and listing DLL files in Pester workflow * Update Pester workflow to use PowerShell commands for copying libraries across platforms * Update Pester workflow to improve library copy commands and list DLL files for macOS ARM * Update Pester workflow to use Unix-style copy command for macOS library * Update Pester and Release workflows to use PowerShell copy commands and adjust library paths for macOS * Fix module import paths for macOS ARM and x64 architectures in AsBuiltReport.Chart.psm1 * Update module version to 0.2.0 and add changelog entry for new features * Update issue templates: enhance bug report and change request forms * Fix validation check in Chart method to compare values count with category names length * Refactor module import statements for macOS architecture and update README with detailed module description and usage examples
* Enhance module manifest with additional metadata including tags, license, project URL, icon, and release notes * Add validation for category names in StackedBarChart and implement tests for chart functions * Update Pester workflow to use actions/checkout@v6 and setup .NET 8.0, streamline build process for MacOS, Linux, and Windows * Enhance Pester workflow to include conditional checks for OS-specific build and copy steps * Add Windows PowerShell build and copy steps to Pester workflow * Enhance Pester and Release workflows for MacOS ARM64 support and update architecture handling in PowerShell module * Remove unnecessary module dependencies from Invoke-Tests script and clean up Pester workflow * Remove conditional code coverage for Windows PowerShell in Pester workflow * Enhance module import statements for macOS ARM and x86 architectures with verbose and debug output * Refactor workflows and issue templates for consistency and clarity; enhance Pester test output with verbose and debug options * Update Pester workflow to list DLL files from macOS ARM directory * Fix path for listing DLL files in macOS ARM directory in Pester workflow * Fix path for listing DLL files in macOS ARM directory in Pester workflow * Update paths for copying libraries and listing DLL files in Pester workflow * Update Pester workflow to use PowerShell commands for copying libraries across platforms * Update Pester workflow to improve library copy commands and list DLL files for macOS ARM * Update Pester workflow to use Unix-style copy command for macOS library * Update Pester and Release workflows to use PowerShell copy commands and adjust library paths for macOS * Fix module import paths for macOS ARM and x64 architectures in AsBuiltReport.Chart.psm1 * Update module version to 0.2.0 and add changelog entry for new features * Update issue templates: enhance bug report and change request forms * Fix validation check in Chart method to compare values count with category names length * Refactor module import statements for macOS architecture and update README with detailed module description and usage examples * Add sample chart images for BarChart, PieChart, and StackedBarChart
…is present (#10) * Initial plan * Fix: Set moderate bar width for single-element stacked bar charts Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com>
…or future enhancements
…gle value scenarios
* Initial plan * Add Signal chart support with New-SignalChart cmdlet Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com> * Add New-SignalChart to FunctionsToExport in module manifest --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com> Co-authored-by: Jonathan Colon <rebelinux@gmail.com>
Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com>
…meTicksBottom Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com>
Sync data
…background color feature Co-authored-by: rebelinux <1002783+rebelinux@users.noreply.github.com>
Added support for setting the background color of the chart.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
1 similar comment
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Fix New-SignalChart to throw terminating error when Values is null
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
There was a problem hiding this comment.
Pull request overview
Adds Signal chart support to the AsBuiltReport.Chart PowerShell module and extends existing chart cmdlets with additional visual customization (background colors + watermark), alongside doc/examples and test updates to support the new functionality and v0.3.0 release.
Changes:
- Introduces
New-SignalChartcmdlet + underlyingSignalChartimplementation. - Adds shared chart features (watermark + figure/data background colors) and expands state reset coverage across cmdlets.
- Updates Pester tests, README/examples, samples, and module metadata for the 0.3.0 release.
Reviewed changes
Copilot reviewed 28 out of 32 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| Todo.md | Adds a new (currently empty) TODO file. |
| Tests/AsBuiltReport.Chart.Functions.Tests.ps1 | Expands Pester coverage for SignalChart, watermarking, format extensions, and stacked-bar validation. |
| Sources/StackedBarChart.cs | Refactors stacked-bar validation and improves single-bar rendering behavior. |
| Sources/SignalChart.cs | Adds the core Signal/Scatter chart rendering implementation. |
| Sources/PowerShell/StackedBarChartPwsh.cs | Adds background/watermark parameters, improves exception type, and resets static state per invocation. |
| Sources/PowerShell/SignalChartPwsh.cs | Adds new New-SignalChart cmdlet wiring chart options into SignalChart. |
| Sources/PowerShell/PieChartPwsh.cs | Adds background/watermark parameters and resets static state per invocation. |
| Sources/PowerShell/BarChartPwsh.cs | Adds background/watermark parameters and resets static state per invocation. |
| Sources/PieChart.cs | Adds background color support + watermark application; improves exception type. |
| Sources/Chart.cs | Adds watermark/background settings, compiled hex regex, secure token generation, and jpg/jpeg extension behavior updates. |
| Sources/BarChart.cs | Adds background color + watermark support and single-value axis limit handling; improves exception type. |
| Sources/AsBuiltReportChart.cs | Updates sample MainEntry usage values/orientation/dimensions. |
| Samples/StackedBarChart.png | Adds updated stacked bar chart sample image. |
| Samples/PieChart.png | Adds updated pie chart sample image. |
| Samples/BarChart.png | Adds updated bar chart sample image. |
| README.md | Rewrites overview and adds usage examples, example index, and watermark documentation. |
| Examples/Example01.ps1 | Adds basic pie chart example script. |
| Examples/Example02.ps1 | Adds advanced pie chart example script. |
| Examples/Example03.ps1 | Adds basic bar chart example script. |
| Examples/Example04.ps1 | Adds advanced bar chart example script. |
| Examples/Example05.ps1 | Adds basic stacked bar chart example script. |
| Examples/Example06.ps1 | Adds advanced stacked bar chart example script. |
| Examples/Example07.ps1 | Adds basic signal chart example script. |
| Examples/Example08.ps1 | Adds DateTime X-axis signal/scatter example script. |
| Examples/Example09.ps1 | Adds multi-line signal/scatter example script. |
| CHANGELOG.md | Documents v0.3.0 additions/changes/fixes. |
| AsBuiltReport.Chart/Src/Assemblies/Core/mac-osx/dummy | Adds placeholder file for macOS Core assemblies directory. |
| AsBuiltReport.Chart/AsBuiltReport.Chart.psm1 | Removes verbose/debug flags from macOS Import-Module paths. |
| AsBuiltReport.Chart/AsBuiltReport.Chart.psd1 | Bumps module version to 0.3.0 and exports New-SignalChart. |
| .github/workflows/Release.yml | Disables tweet and Bluesky release-post jobs. |
| .github/ISSUE_TEMPLATE/change_request.yml | Adds default assignee. |
| .github/ISSUE_TEMPLATE/bug_report.yml | Adds default assignee. |
Comments suppressed due to low confidence (1)
Todo.md:2
- Todo.md was added but is effectively empty. Either remove this file from the PR or populate it with actual TODO items so it doesn’t create noise in the repo history.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
1 similar comment
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [0]
|
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [1]
|
4 similar comments
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [1]
|
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [1]
|
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [1]
|
|
PSScriptAnalyzer results: Errors: [0], Warnings: [35], Information: [1]
|
[0.3.0] - 2026-03-14
Added
Changed
Update module version to 0.3.0
Fixed