Skip to content

refactor: refactor ParsePlaceholders for conciseness and adaptability#32

Open
TeCHiScy wants to merge 1 commit into
lukasjarosch:developfrom
TeCHiScy:runspan
Open

refactor: refactor ParsePlaceholders for conciseness and adaptability#32
TeCHiScy wants to merge 1 commit into
lukasjarosch:developfrom
TeCHiScy:runspan

Conversation

@TeCHiScy

Copy link
Copy Markdown
Contributor
  • avoid panic with corrupted placeholder
  • support any run spans between delimeters
  • update template.docx to include corrupted and nested cases
  • lay foundation for a template engine to be introduced

closes #29

@TeCHiScy

Copy link
Copy Markdown
Contributor Author
  • WIP, do not merge now.
  • hi @lukasjarosch, seems the lint ci failed, is it related to the go version / runner itself?

- avoid panic with corrupted placeholder
- support any run spans between delimeters
- update `template.docx` to include corrupted and nested cases
- lay foundation for a template engine to be introduced

closes lukasjarosch#29
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.

Panic in ParsePlaceholders function in placeholders.go

1 participant