Skip to content

Refactor import statement for chunking service#51

Merged
brunoalbin23 merged 1 commit intomainfrom
fix/pipeline-again
Dec 17, 2025
Merged

Refactor import statement for chunking service#51
brunoalbin23 merged 1 commit intomainfrom
fix/pipeline-again

Conversation

@JuanPalbo
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 17, 2025 00:55
@JuanPalbo JuanPalbo closed this Dec 17, 2025
Copy link
Copy Markdown
Contributor

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 simplifies the import statement for the chunking service function by removing an unnecessary alias. The function document_to_chunks already exists with its actual name in the chunking_service module, so the previous aliasing from split_documents was redundant.

Key Changes:

  • Removed import alias to use the function's actual name directly, aligning with the naming pattern used for other service imports in the file (pdf_to_document, chunks_to_embeddings)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

🔍 PR Validation Results

Check Status
Build ✅ success
Trivy Check Security tab

View detailed results

@JuanPalbo JuanPalbo reopened this Dec 17, 2025
@JuanPalbo
Copy link
Copy Markdown
Collaborator Author

Jijo

@github-actions
Copy link
Copy Markdown

🔍 PR Validation Results

Check Status
Build ✅ success
Trivy Check Security tab

View detailed results

@brunoalbin23 brunoalbin23 merged commit 6bc5f3c into main Dec 17, 2025
13 checks passed
@brunoalbin23 brunoalbin23 deleted the fix/pipeline-again branch December 17, 2025 01:07
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.

3 participants