Skip to content

Conversation

@CahidArda
Copy link
Collaborator

No description provided.

@linear
Copy link

linear bot commented Aug 28, 2025

@ytkimirti ytkimirti requested a review from Copilot September 10, 2025 09:45
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 label parameter support to the QStash SDK, enabling users to assign labels to messages and schedules for better log filtering and organization.

  • Adds optional label parameter to all message publishing and scheduling methods
  • Extends data models to include label fields for messages, schedules, logs, and DLQ entries
  • Implements comprehensive test coverage for all label functionality across sync and async APIs

Reviewed Changes

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

Show a summary per file
File Description
qstash/message.py Adds label parameter to message APIs and data models
qstash/schedule.py Adds label parameter to schedule APIs and Schedule model
qstash/log.py Adds label field to Log model and LogFilter for filtering
qstash/dlq.py Adds label field to DlqMessage model and DlqFilter
qstash/asyncio/message.py Adds label parameter to async message APIs
qstash/asyncio/schedule.py Adds label parameter to async schedule APIs
tests/test_message.py Tests label functionality for message operations
tests/test_schedules.py Tests label functionality for schedule operations
tests/asyncio/test_message.py Tests label functionality for async message operations
tests/asyncio/test_schedules.py Tests label functionality for async schedule operations

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

@CahidArda CahidArda merged commit 35ff096 into main Sep 12, 2025
1 check failed
@CahidArda CahidArda deleted the DX-2069-label branch September 12, 2025 16:58
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