Skip to content

Fix .hbs file copyright parsing by supporting indented content#188

Merged
CreatorHead merged 1 commit intomainfrom
fix-hbs-copyright-parsing
Feb 3, 2026
Merged

Fix .hbs file copyright parsing by supporting indented content#188
CreatorHead merged 1 commit intomainfrom
fix-hbs-copyright-parsing

Conversation

@CreatorHead
Copy link
Contributor

  • Add support for handlebars (.hbs) files with {{! ... }} multi-line comments
  • Copyright lines in .hbs files are indented with spaces but lack direct comment prefixes
  • Added ' ' (two spaces) to commentPrefixes array to handle indented content within multi-line comments
  • Added comprehensive test coverage for handlebars copyright parsing
  • All existing tests continue to pass

Revert copyright year change in update.go

  • Change back from 2022, 2026 to 2023, 2026

🛠️ Description

🔗 External Links

👍 Definition of Done

  • New functionality works?
  • Tests added?

🤔 Can be merged upon approval?

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

- Add support for handlebars (.hbs) files with {{! ... }} multi-line comments
- Copyright lines in .hbs files are indented with spaces but lack direct comment prefixes
- Added '  ' (two spaces) to commentPrefixes array to handle indented content within multi-line comments
- Added comprehensive test coverage for handlebars copyright parsing
- All existing tests continue to pass

Revert copyright year change in update.go

- Change back from 2022, 2026 to 2023, 2026
@CreatorHead CreatorHead requested a review from a team as a code owner February 3, 2026 07:20
Copy link
Collaborator

@mohanmanikanta2299 mohanmanikanta2299 left a comment

Choose a reason for hiding this comment

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

LGTM

@CreatorHead CreatorHead merged commit 8df03b7 into main Feb 3, 2026
5 checks passed
@CreatorHead CreatorHead deleted the fix-hbs-copyright-parsing branch February 3, 2026 10:48
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