Skip to content

Conversation

@EvilLord666
Copy link
Member

During this PR were implemented the following:

  1. Multiple curly bracket proper processing (Process multiple curly brackets #21 )
  2. Changed algorithm to use state machine for template processing (Use statemachine 4 input processing #1)

@EvilLord666 EvilLord666 requested a review from stdstring June 11, 2025 19:19
@stdstring
Copy link

Наверное мое самое большое замечание: код в formatter.go просто больно читать - я бы вынес часть кода из функций форматирования в виде внутренних функций (как вариант - создал бы дополнительную структуру и связал бы эти функции с ней)

@stdstring
Copy link

stdstring commented Jun 25, 2025

Ну и да - сделал бы частичный или полный squash коммитов

@EvilLord666
Copy link
Member Author

Наверное мое самое большое замечание: код в formatter.go просто больно читать - я бы вынес часть кода из функций форматирования в виде внутренних функций (как вариант - создал бы дополнительную структуру и связал бы эти функции с ней)

Да, для того чтобы sf работал быстро, код в нем именно такой, пожалуй раз замечания не критичны я наверное не буду дополнительно что-то реализовывать

@EvilLord666 EvilLord666 merged commit 30d1409 into develop Jun 26, 2025
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.

3 participants