Skip to content

Fix: MilkdropShader - ignore code comments (#958 / v4.1.x)#977

Merged
kblaschke merged 1 commit intoprojectM-visualizer:v4.1.xfrom
mbaetgen-wup:bugfix/handle-parser-comments
Mar 30, 2026
Merged

Fix: MilkdropShader - ignore code comments (#958 / v4.1.x)#977
kblaschke merged 1 commit intoprojectM-visualizer:v4.1.xfrom
mbaetgen-wup:bugfix/handle-parser-comments

Conversation

@mbaetgen-wup
Copy link
Copy Markdown
Collaborator

@mbaetgen-wup mbaetgen-wup commented Mar 5, 2026

Fixes #958

@mbaetgen-wup mbaetgen-wup force-pushed the bugfix/handle-parser-comments branch from f246ce7 to f890b8a Compare March 5, 2026 05:33
@mbaetgen-wup mbaetgen-wup changed the title Fix: MilkdropShader - ignore code comments (#958) Fix: MilkdropShader - ignore code comments (#958 / 4.1.x) Mar 5, 2026
@mbaetgen-wup mbaetgen-wup added the bug The issue is (potentially) a bug. label Mar 5, 2026
@mbaetgen-wup mbaetgen-wup added this to the 4.1.7 milestone Mar 5, 2026
@mbaetgen-wup mbaetgen-wup force-pushed the bugfix/handle-parser-comments branch from f890b8a to d5a87cc Compare March 5, 2026 05:59
@mbaetgen-wup mbaetgen-wup changed the title Fix: MilkdropShader - ignore code comments (#958 / 4.1.x) Fix: MilkdropShader - ignore code comments (v4.1.x) Mar 5, 2026
@mbaetgen-wup mbaetgen-wup changed the title Fix: MilkdropShader - ignore code comments (v4.1.x) Fix: MilkdropShader - ignore code comments (#958 / v4.1.x) Mar 5, 2026
Copy link
Copy Markdown
Member

@kblaschke kblaschke left a comment

Choose a reason for hiding this comment

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

LGTM, also great to see additional tests!

@kblaschke kblaschke force-pushed the bugfix/handle-parser-comments branch from d5a87cc to 55fb034 Compare March 30, 2026 09:11
@kblaschke kblaschke merged commit 55fb034 into projectM-visualizer:v4.1.x Mar 30, 2026
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug The issue is (potentially) a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants