Skip to content

Issue #171 Fix src/library/colorizer tests #263

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

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

tsayukov
Copy link
Collaborator

@tsayukov tsayukov commented Jul 16, 2024

Issue #171

В colors.h добавил объявление оператора << для NColorizer::EAnsiCode, так как без него вызывался оператор << для char (underlying тип енума кастился в char по всей видимости).

@tsayukov tsayukov requested a review from Gazizonoki July 16, 2024 11:35
@tsayukov tsayukov force-pushed the issue-#171-fix-src/library/colorizer-tests branch from c4e28dc to cbe2729 Compare July 17, 2024 08:27
@Gazizonoki
Copy link
Collaborator

Gazizonoki commented Jul 25, 2024

Прошу прощения за задержку, у нас произошла небольшая смена курса в переносе C++ SDK, util и library мы скорее всего откатим, их неудобно поддерживать в разных состояниях внутри и снаружи, поэтому пока что возвращаем их обратно и PR'ы в этот код не принимаем. Как закончим работу, отпишусь тут, если все еще актуально

@tsayukov
Copy link
Collaborator Author

Прошу прощения за задержку, у нас произошла небольшая смена курса в переносе C++ SDK, util и library мы скорее всего откатим, их неудобно поддерживать в разных состояниях внутри и снаружи, поэтому пока что возвращаем их обратно и PR'ы в этот код не принимаем. Как закончим работу, отпишусь тут, если все еще актуально

Привет! Спасибо, что сообщил! Я буду периодически заглядывать узнать, как здесь дела, но если будут ишью, которые можно сделать, пиши, всегда буду рад помочь

@Gazizonoki
Copy link
Collaborator

Прошу прощения за задержку, у нас произошла небольшая смена курса в переносе C++ SDK, util и library мы скорее всего откатим, их неудобно поддерживать в разных состояниях внутри и снаружи, поэтому пока что возвращаем их обратно и PR'ы в этот код не принимаем. Как закончим работу, отпишусь тут, если все еще актуально

Привет! Спасибо, что сообщил! Я буду периодически заглядывать узнать, как здесь дела, но если будут ишью, которые можно сделать, пиши, всегда буду рад помочь

Закончили работу над возвращением util и library, если есть желание, можешь продолжить чинить тесты. Однако править код library и util теперь нельзя, только CMakeLists, этот код будет приезжать из внутреннего репозитория, если нужно будет сделать правку из-за бага или нерабочего теста, приноси их мне, я буду мержить их у нас внутри. Также возможно в ближайшем будущем принесу еще пару задач по SDK.

@tsayukov tsayukov force-pushed the issue-#171-fix-src/library/colorizer-tests branch from cbe2729 to d6ffd0a Compare August 6, 2024 16:20
@tsayukov
Copy link
Collaborator Author

tsayukov commented Aug 8, 2024

@Gazizonoki

@Gazizonoki Gazizonoki merged commit b53c38c into main Aug 8, 2024
7 checks passed
@Gazizonoki Gazizonoki deleted the issue-#171-fix-src/library/colorizer-tests branch August 8, 2024 16:23
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