N
и на сколько сильно вся эта поддержка ie тормозит нормальные браузеры, ведь туда весь этот хлам тоже складывается?
Size: a a a
2021 April 21
ОЭ
Подскажите, первый раз делаю контрибьют в open source. Добавил в библиотеку две новые фичу. Как лучше оформить пулл реквест? В два разных или в один?
N
Есть такое подозрение, что
@babel/polyfill надо в сборку подключатьА
Нормальный полифилл смотрит если в броврезе есть реализация, то не добавляет ничего, кажется так
N
Скорее всего в два лучше, если возможно
N
он у меня и там есть, просто на 2х разных видео уроках по разному
N
Видно же, что его тут нет 🤔
ОЭ
А как лучше их описать? Указать в комментарии какие строчки изменил и добавил и описать почему? Или только в общих словах описать что было сделано?
N
ой, не то скинул
N
Какие изменил строчки и так видно будет. Лучше описать человеческим языком, что сделал
ОЭ
N
Пожалуйста
N
а есть пак, в котором вообще все полифилы есть, или так не делают?
N
и что б он сам понял, какой полифил нужно взять
N
Не встречал такого
ОЭ
А если я в процессе добавления новой фичи поправил некритичный баг, который влияет на новую фичу. Мне этот баг оформлять в отдельный пулл реквест?
IS
поищите в репе, обычно пишут что и как они хотят видеть
N
И кстати
ОЭ
Репа маленькая, там всего 4 issue и все pull request только от dependencybot. Там нет описания как они хотят это все видеть
N
Если, например, поправил несколько багов, то можно и в один всё пихнуть.
А если одно — поправил баг, а другое — новая фича, то отдельно)
А если одно — поправил баг, а другое — новая фича, то отдельно)