Skip to content

Conversation

quge009
Copy link
Collaborator

@quge009 quge009 commented Sep 11, 2025

  • rename kql doc directory for improved clarity
  • remove unused sql files

@Copilot Copilot AI review requested due to automatic review settings September 11, 2025 08:24
Copy link

@Copilot 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 cleans up data organization by renaming the KQL document directory from "demoM3_LTP" to "LTP" for improved clarity and removing unused SQL configuration and files.

  • Removes the SQLDB_DIR configuration constant and replaces it with inline path construction
  • Updates KQL document paths from "demoM3_LTP" to "LTP" for better naming consistency
  • Removes unused SQL-related files and build script copying operation

Reviewed Changes

Copilot reviewed 8 out of 25 changed files in this pull request and generated no comments.

File Description
src/copilot-chat/src/copilot_agent/utils/sql.py Replaces SQLDB_DIR import with inline path construction using DATA_DIR
src/copilot-chat/src/copilot_agent/ltp/ltp_dashboard.py Updates KQL document paths from "demoM3_LTP" to "LTP" directory
src/copilot-chat/src/copilot_agent/config.py Removes SQLDB_DIR configuration constant
src/copilot-chat/build/build-pre.sh Removes copying operation for dashboard tables

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@quge009 quge009 requested a review from yukirora September 11, 2025 08:41
@quge009 quge009 merged commit fb52875 into dev Sep 12, 2025
1 check passed
@quge009 quge009 deleted the lequ/copilot/minor-fix branch September 12, 2025 00:37
This was referenced Sep 15, 2025
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