Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow prefix and suffix with double braces in label format #446

Merged

Conversation

TomW1605
Copy link
Contributor

No description provided.

@Donkie
Copy link
Owner

Donkie commented Aug 26, 2024

Thank you, I think it would be good to add some help somewhere on what can be achieved now. Either by modifying the existing default template or adding some help text. Right now all I see is a bunch of magic regex in the code.

@TomW1605
Copy link
Contributor Author

thats fair enough, i will add some help text and add an example to the default template

@TomW1605
Copy link
Contributor Author

i have added some examples to the default template and added an explanation to the help text. it does make the help text longer so feel free to re word it if you want

@Donkie
Copy link
Owner

Donkie commented Sep 8, 2024

Looks good, thank you!

@Donkie Donkie merged commit a826db1 into Donkie:master Sep 8, 2024
12 checks passed
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