hound-search/hound

Add a Github CI check to make sure that if an asset is edited, ui/bindata.go is updated

Open

#365 aperta il 30 ott 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (606 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

Star
 (5470 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

We serve static assets from a serialized file in order to make Hound go-get-able (see #7). Confirming that UI PRs come with an update to ui/bindata.go would be helpful for future PRs.

Here's some helpful context (thanks to @kellegous for a lot of this):

  1. ui/bindata.go contains all the contents from ui/assets/* and is generated here in the Makefile.
  2. Generating ui/bindata.go as part of the install (i.e. in //go:generate) might be finicky on some systems, so sticking with manually updating it is a safer play for now.
  3. As mentioned above, Issue #7 has more context for why we use this approach.
  4. If you're curious, assets are read in from ui/bindata.go through calls to Asset in ui#serveAsset.

Guida contributor

Direzione di ricerca
Crea un flusso di lavoro GitHub Actions che verifichi se ui/bindata.go viene aggiornato ogni volta che i file in ui/assets/ vengono modificati.
Tech stack
gojavascript
Dominio
ci cdbuild system
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitGitHub Actions
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70