Skip to content

Refactor code structure for improved readability and maintainability#67

Merged
yatharthk2 merged 3 commits intomainfrom
yatharth/pipecat-patch
Mar 24, 2026
Merged

Refactor code structure for improved readability and maintainability#67
yatharthk2 merged 3 commits intomainfrom
yatharth/pipecat-patch

Conversation

@yatharthk2
Copy link
Collaborator

@yatharthk2 yatharthk2 commented Mar 24, 2026

Pull Request Checklist

Please ensure that your PR meets the following requirements:

  • I have read the CONTRIBUTING guide.
  • I have updated the documentation (if applicable).
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

Description

Patches the Pipecat-moss to be compatible with the latest inferedge-moss version.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Open with Devin

Copilot AI review requested due to automatic review settings March 24, 2026 22:34
@vercel
Copy link
Contributor

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
moss_vitepress Ready Ready Preview, Comment Mar 24, 2026 11:16pm
moss-samples Ready Ready Preview, Comment Mar 24, 2026 11:16pm

Request Review

This comment was marked as resolved.

Copy link
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

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.


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

@ashvathsureshkumar ashvathsureshkumar self-requested a review March 24, 2026 23:23
requires-python = ">=3.10"
requires-python = ">=3.10,<3.14"
dependencies = [
"inferedge-moss>=1.0.0b18",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

19?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have allowed b19, its working with python 3.13

@@ -3,7 +3,7 @@ name = "pipecat-moss"
version = "0.0.3"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bumping the version? and add a changelog?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.0.3 was never published, I hit an error, so this would go under 0.0.3

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@yatharthk2 yatharthk2 merged commit eddc20c into main Mar 24, 2026
19 checks passed
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