Skip to content

fix: source - #567

Merged
bashbunni merged 5 commits into
charmbracelet:mainfrom
rsteube:fix-source
Mar 26, 2025
Merged

bashbunni merged 5 commits into
charmbracelet:mainfrom
rsteube:fix-source

Conversation

@rsteube

@rsteube rsteube commented Jan 4, 2025 •

Copy link
Copy Markdown
Contributor

Expands the parsed Source command instead of re-parsing it later.

Works for me so far, but not sure if this is the right approach.

fix #566

@rsteube
rsteube marked this pull request as ready for review January 4, 2025 23:25
@caarlos0

caarlos0 commented Jan 6, 2025

Copy link
Copy Markdown
Contributor

does this do the same as #551 ?

@rsteube

rsteube commented Jan 6, 2025

Copy link
Copy Markdown
Contributor Author

Oh, didn't see that one. I think so, but let me see.
Looks like the logged prefix might get lost there though.

This one is a bit tricky, so better have a closer look.

@caarlos0
caarlos0 requested a review from a team as a code owner March 26, 2025 16:33
@caarlos0
caarlos0 requested review from bashbunni and removed request for a team March 26, 2025 16:33

@caarlos0 caarlos0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just tested it, seems to work! thanks for the pr!

i have one small remark only :)

Comment thread parser/parser.go Resolved
@caarlos0 caarlos0 changed the title fix source fix: source Mar 26, 2025
rsteube and others added 3 commits March 26, 2025 17:57
Co-authored-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the bug Something isn't working label Mar 26, 2025
@caarlos0 caarlos0 assigned caarlos0 and unassigned caarlos0 Mar 26, 2025

@bashbunni bashbunni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, thank you!

@bashbunni
bashbunni merged commit fd38bed into charmbracelet:main Mar 26, 2025
caarlos0 added a commit that referenced this pull request Mar 27, 2025
@caarlos0

Copy link
Copy Markdown
Contributor

see also #604

leosunmo added a commit to leosunmo/vhs that referenced this pull request Oct 3, 2025
Co-authored-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
melMass pushed a commit to melMass/vhs that referenced this pull request Nov 17, 2025
Co-authored-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Source: format settings have no effect

4 participants

⌂ 换网址