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.

Sign up for GitHub

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

Jump to bottom

Use Template for Formatting the Issues #75

Closed
notJoon opened this issue Sep 27, 2024 · 0 comments · Fixed by #98
Closed

Use Template for Formatting the Issues #75

notJoon opened this issue Sep 27, 2024 · 0 comments · Fixed by #98
Labels
good first issue Good for newcomers T-format Type: related with formatter

Comments

@notJoon
Copy link
Contributor

notJoon commented Sep 27, 2024

Description

The current lint results are assembled using builder pattern functions, but this method has a lot of duplicate code and becomes unnecessarily complex when trying to insert new formatting components.

Therefore, it seems necessary to improve the formatting approach by using a package like text/template.
@notJoon notJoon added good first issue Good for newcomers T-format Type: related with formatter labels Sep 27, 2024
@0xtekgrinder 0xtekgrinder mentioned this issue Oct 24, 2024
Merged
notJoon pushed a commit that referenced this issue Oct 30, 2024
@0xtekgrinder
refactor: use text templates for formatting issues (#98)
dba079d 
fixes #75

This PR refactor how the formatter works to use text/template go
package.

This replicates so far how the old formatter is working
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers T-format Type: related with formatter
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: use text templates for formatting issues 0xtekgrinder/tlin
1 participant
@notJoon