search

ydb-platform / ydb-cpp-sdk

YDB C++ SDK
Apache License 2.0
11 stars 11 forks source link

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

Closed tsayukov closed 3 months ago

tsayukov commented 3 months ago

Issue #171

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

Gazizonoki commented 3 months ago

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

tsayukov commented 3 months ago

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

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

Gazizonoki commented 3 months ago

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

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

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

tsayukov commented 3 months ago

@Gazizonoki