The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Выпуск nomenus-rex 0.7.0, утилиты для массового переименования файлов , opennews (?), 20-Июл-22, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


2. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  –1 +/
Сообщение от Онаним (?), 20-Июл-22, 23:28 
Вопрос снят. Посмотрел коммит. Не, ребята, я мимо. Не вложенные циклы конечно, но...
Ответить | Правка | Наверх | Cообщить модератору

4. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  +2 +/
Сообщение от Аноним (4), 20-Июл-22, 23:40 
можно пожалуйста ссылку для особо ленивых?
Ответить | Правка | Наверх | Cообщить модератору

8. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  +/
Сообщение от topin89 (ok), 21-Июл-22, 00:25 
Держи: https://github.com/ANGulchenko/nomenus-rex/commit/6185d5914a...
Ответить | Правка | Наверх | Cообщить модератору

26. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  –2 +/
Сообщение от pashev.ru (?), 21-Июл-22, 10:12 
Changelog нельзя изменять в одном коммите с кодом. Я запрещаю и проклинаю всех, кто так делает.
Ответить | Правка | Наверх | Cообщить модератору

27. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  +/
Сообщение от A.Stahl (ok), 21-Июл-22, 10:46 
А какая выгода от изменения ченджлога отдельным коммитом от кода, который этот ченджлог описывает?

Кто-то читает диффы ченджлогов? Ну, в данном случае диффы всегда сверху файла.

Ответить | Правка | Наверх | Cообщить модератору

33. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  +2 +/
Сообщение от topin89 (ok), 21-Июл-22, 12:55 
Черри-пикать удобнее. Из-за этого в коммитах лучше держать минимальные изменения, после которых код может компилироваться/выполняться. Ну и временами читать, как код менялся, удобнее по маленьким коммитам.

Диффы читают на ревью PR'ов. Может не всегда, но время от времени так удобнее.

Ответить | Правка | Наверх | Cообщить модератору

9. "Выпуск nomenus-rex 0.7.0, утилиты для массового переименован..."  +1 +/
Сообщение от topin89 (ok), 21-Июл-22, 00:35 
Зря ты так. Видно, что человек учится, набивает шишки. Уже понял, что юникод -- это магия и пусть специальная библиотека с этим возится. Даже понял, что вложенные циклы в 1000 раз хуже, чем использование специальных структур для поиска.

Зачем делать новость из софта уровня "я сделаль" я не понимаю, но автору кода желаю учиться и развиваться, а также читать документацию. Та же проверка, что вставка в map удалась через проверку размера карты когда тот же insert уже выдаёт, прошла вставка или нет -- это неправильно.

Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру