Skip to content

[claude skill workflow testing] Add optional comment parameter to concat::fragment#832

Closed
mehul-jain1 wants to merge 1 commit intomainfrom
MODULES-11715
Closed

[claude skill workflow testing] Add optional comment parameter to concat::fragment#832
mehul-jain1 wants to merge 1 commit intomainfrom
MODULES-11715

Conversation

@mehul-jain1
Copy link
Copy Markdown
Contributor

Summary

  • Adds an optional comment parameter to concat::fragment that prepends # -prefixed comment lines above the fragment content
  • Supports multiline comments (each line gets the # prefix)
  • Validates that comment cannot be used with source (only with content)
  • Includes 4 new unit tests covering: basic usage, multiline, source rejection, and type validation

Test plan

  • bundle exec rake syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop — all pass
  • bundle exec rake parallel_spec — 258 examples, 0 failures
  • CI pipeline passes
  • Acceptance tests on provisioned target (if applicable)

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mehul-jain1 mehul-jain1 requested review from a team, b4ldr, bastelfreak, ekohl and smortex as code owners April 7, 2026 05:34
@mehul-jain1 mehul-jain1 requested a review from Copilot April 7, 2026 05:34
@mehul-jain1 mehul-jain1 marked this pull request as draft April 7, 2026 05:34
@mehul-jain1 mehul-jain1 review requested due to automatic review settings April 7, 2026 05:35
@mehul-jain1 mehul-jain1 changed the title [MODULES-11715] Add optional comment parameter to concat::fragment [claude skill workflow testing] Add optional comment parameter to concat::fragment Apr 7, 2026
@skyamgarp
Copy link
Copy Markdown
Contributor

Closing as it's Testing PR

@skyamgarp skyamgarp closed this Apr 9, 2026
@skyamgarp skyamgarp deleted the MODULES-11715 branch April 9, 2026 07:52
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.

2 participants