Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: Update LUA and multiline #312

Merged
merged 1 commit into from
Feb 3, 2025
Merged

fix: Update LUA and multiline #312

merged 1 commit into from
Feb 3, 2025

Conversation

obs-gh-justindaines
Copy link
Contributor

Update LUA and multiline to handle stdout F and stdout P

@obs-gh-justindaines
fix: Update LUA and multiline
17e6802 
Update LUA and multiline to handle `stdout F` and `stdout P`
@obs-gh-justindaines obs-gh-justindaines requested a review from a team February 3, 2025 15:45
obs-gh-felipequiros
obs-gh-felipequiros approved these changes Feb 3, 2025
View reviewed changes
Copy link

@obs-gh-felipequiros obs-gh-felipequiros left a comment

Choose a reason for hiding this comment

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

Looks good. Change captures stdout P and F.

@obs-gh-justindaines obs-gh-justindaines merged commit 7de62d2 into main Feb 3, 2025
11 checks passed
@obs-gh-justindaines obs-gh-justindaines deleted the jdaines/multi-line-stdout-fix branch February 3, 2025 15:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@obs-gh-felipequiros obs-gh-felipequiros obs-gh-felipequiros approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@obs-gh-justindaines @obs-gh-felipequiros