Skip to content

Conversation

Copy link

Copilot AI commented Dec 16, 2025

Executable scripts at repository root pollute the package namespace for library projects. Moved create_sample_data.py to the examples/ directory where it belongs alongside other demo scripts.

Changes

  • Relocated create_sample_data.pyexamples/create_sample_data.py
  • Updated ruff exclusion path in pyproject.toml

Script behavior unchanged—still generates sample_qa_data.json in working directory when executed.

Original prompt

This section details on the original issue you should resolve

<issue_title>Move root-level create_sample_data.py under examples/scripts</issue_title>
<issue_description>## Summary
The repository root contains create_sample_data.py, which is an example/helper script. Keeping executable scripts at the root is bad practice for a library project; they should live under examples/ or scripts/ and not be part of the package import path.

Expected

  • Relocate create_sample_data.py to examples/ (or scripts/), keeping behavior the same.
  • Update any references in docs/README to point to the new location.

Suggested fix

  • Move the file to examples/create_sample_data.py (or similar).
  • Adjust documentation to reference the new path.

Why it matters

Keeping runnable scripts out of the repo root makes the project structure cleaner and reduces confusion for library consumers.
</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Move create_sample_data.py under examples/scripts Move create_sample_data.py from root to examples/ Dec 16, 2025
Copilot AI requested a review from chicham December 16, 2025 15:01
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.

Move root-level create_sample_data.py under examples/scripts

2 participants