Skip to content

Comments

Remove hardcoded project-specific data from report template#13

Merged
makomweb merged 2 commits intonextfrom
copilot/sub-pr-10-another-one
Jan 25, 2026
Merged

Remove hardcoded project-specific data from report template#13
makomweb merged 2 commits intonextfrom
copilot/sub-pr-10-another-one

Conversation

Copy link
Contributor

Copilot AI commented Jan 25, 2026

The resources/report/report.js file contained hardcoded example data from "src/TBone/" that was committed during development. This file is dynamically generated by ReportCommand at runtime and should not exist in the repository.

Changes

  • Removed resources/report/report.js from repository
  • Template files (chart.js, styles.css, index.html) remain as they contain no project-specific data

The command generates report.js in the output directory:

// ReportCommand.php line 219
file_put_contents($destination, 'const reportData = '.$json.';');

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: makomweb <1567373+makomweb@users.noreply.github.com>
Copilot AI changed the title [WIP] Update CLI report generation command based on feedback Remove hardcoded project-specific data from report template Jan 25, 2026
Copilot AI requested a review from makomweb January 25, 2026 14:48
@makomweb makomweb marked this pull request as ready for review January 25, 2026 15:06
@makomweb makomweb merged commit 751ad09 into next Jan 25, 2026
@makomweb makomweb deleted the copilot/sub-pr-10-another-one branch January 25, 2026 15:11
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