-
Notifications
You must be signed in to change notification settings - Fork 9
Баг на странице прогресса перевода #40
Comments
Сейчас всё изменится. |
Так, не сейчас, а где-то через полчаса ) Где-то до migration сейчас всё показывается верно, дальше расхождения. Но номера коммитов верные, можно по ним делать дифф. |
Похоже опять все сломалось ;) |
Да, кажется этот кусок надо будет переписать. Уже 2 проблемы в этой части логики. |
В теории этот баг будет правиться? По идее хорошо бы уже начать работать над переводом документации 9.х |
Да, надо бы постараться вернуться к нормальной жизни в ближайшее время. |
Сорь, я что-то совсем закопался в последнее время =( Постараюсь на этих выходных заняться. Главное чтоб из головы не вылетело. |
Изначально была проблема в том, что диффы подсчитываются довольно быстро, т.к. сгружаются отдельной историей изменений. А вот с обновлениями проблемы, т.к. оно для каждого файла проверяет и актуализирует их, вместо того, чтобы просто смотреть таймстамп изменения. В результате все файлы вообще всех веток сгружались в оперативку и она просто кончалась. Поэтому синхронизация самой документации отваливалась, а история и дифф изменений отрабатывали. Короче, пока что просто снял ограничения на допустимую оперативу, теперь всё должно нормально синкаться. Но саму логику пока не переписывал, решил вначале апнуть ларку и некоторые устаревшие пакеты. P.S. Это неделю назад, если что, примерно когда и дизайн ещё чуть подправил. Просто забыл отписаться тогда в ишью этом. |
Странное поведение сейчас наблюдается на странице https://laravel.su/status/8.x после обновления данных. Коммит один и тот же, но пишет что отстает на 2 коммита. И такая же ситуация у многих других актуальных файлов
The text was updated successfully, but these errors were encountered: