Skip to content

Conversation

@pablolagreca
Copy link
Contributor

@pablolagreca pablolagreca commented Jun 13, 2025

Copy link
Contributor

@silaslenihan silaslenihan left a comment

Choose a reason for hiding this comment

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

Will there be a followup PR for tests for this new functionality?

@pablolagreca pablolagreca marked this pull request as ready for review June 17, 2025 21:22
@pablolagreca pablolagreca requested a review from a team as a code owner June 17, 2025 21:22
dhaidashenko
dhaidashenko previously approved these changes Jun 18, 2025
@pablolagreca
Copy link
Contributor Author

Will there be a followup PR for tests for this new functionality?

Will there be a followup PR for tests for this new functionality?

Yes, there's a few test cases missing. I'll add them later on. Tracking the action item in PLEX-1524

ilija42
ilija42 previously approved these changes Jun 18, 2025
@pablolagreca pablolagreca dismissed stale reviews from ilija42 and dhaidashenko via 689a284 June 19, 2025 15:17
dhaidashenko
dhaidashenko previously approved these changes Jun 20, 2025
silaslenihan

This comment was marked as outdated.

dhaidashenko
dhaidashenko previously approved these changes Jun 26, 2025
dhaidashenko
dhaidashenko previously approved these changes Jun 27, 2025
ilija42
ilija42 previously approved these changes Jun 27, 2025
dhaidashenko
dhaidashenko previously approved these changes Jun 27, 2025
ilija42
ilija42 previously approved these changes Jun 27, 2025
@pablolagreca pablolagreca merged commit e94d470 into main Jun 27, 2025
22 checks passed
@pablolagreca pablolagreca deleted the PLEX-250 branch June 27, 2025 11:46
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.

4 participants