search

slovensko-digital / navody-frontend

navody.digital frontend je knižnica komponentov, ktorá výchadza z open source knižnice GOV.UK Frontend a pridáva k nej nové komponenty.
https://frontend.staging.navody.digital/
MIT License
7 stars 17 forks source link

Upstream sync #128

Closed Zippersk closed 3 years ago

Zippersk commented 3 years ago

Fix @allie47 MR https://github.com/slovensko-digital/navody-frontend/pull/127 build.

Zippersk commented 3 years ago

Lokálne mi build funguje po "oprave" copyright testu: https://github.com/slovensko-digital/navody-frontend/blob/2e1562c13cfe4b7a7460aaaf0bbff3fa3b8a1143/tasks/gulp/__tests__/after-build-dist.test.js#L64-L67

Copyright sa do CSS nepridáva, pretože máme zakomentované Font-Faces: https://github.com/slovensko-digital/navody-frontend/blob/2e1562c13cfe4b7a7460aaaf0bbff3fa3b8a1143/package/govuk/helpers/_all.scss#L3-L7

Asi by bolo najlepšie keby si @allie47 mergne tento MR do svojho MR (je tam lepší popis čo v MR urobila), následne sa tento môže zatvoriť a @allie47's mergnúť.

čo ty nato @jsuchal ?

allie47 commented 3 years ago

@Zippersk Co presne brani tomu, aby sa proste zmenil target tohto MR a targetovalo to moju branchu instead? Tak vieme jednoducho rovno mergenut tento MR do nej a spravit to transparentne (nie tym ze ja to tam napcham manualnym mergeom)?

(ak si nahodou myslel rovno toto, my apologies)

Zippersk commented 3 years ago

@allie47 má pravdu. Je určite jednoduchšie urobiť MR v jej repe :) Takže tento zatváram a otvoril som https://github.com/allie47/navody-frontend/pull/1

allie47 commented 3 years ago

@Zippersk Dockam na vyjadrenie od @jsuchal nez to mergenem este