Skip to content

Conversation

@klei22
Copy link
Collaborator

@klei22 klei22 commented Oct 20, 2025

This pull request enhances the compile_report.sh script by adding an optional cleanup feature and improving argument handling. The most important changes are:

New Features:

  • Added support for a --clean (or -c) flag, which, when specified, will remove all archived items after creating the report archive. This helps automate cleanup of temporary or intermediate files. [1] [2]

Argument Handling Improvements:

  • Improved argument parsing to separate user-provided files/directories from flags, ensuring that only valid items are included in the archive and that flags like --clean are not mistakenly treated as files. [1] [2]

Safety Enhancements:

  • During cleanup, the script now checks if an item is tracked by git and skips deletion of such items, preventing accidental removal of source-controlled files.

@klei22 klei22 requested review from Copilot and gkielian October 20, 2025 04:02
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

This PR adds an optional cleanup feature to the compile_report.sh script that allows users to automatically remove archived items after creating the report archive. It also improves argument handling to properly separate user-provided files and directories from command-line flags.

  • Added --clean/-c flag support for automatic cleanup of archived items
  • Improved argument parsing to distinguish between files/directories and command flags
  • Added safety checks to prevent deletion of git-tracked files during cleanup

Tip: Customize your code reviews with copilot-instructions.md. Create the file or 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