Skip to content

Commit

Permalink
Merge pull request #246 from jhult/fix-emoji-failfast
Browse files Browse the repository at this point in the history
Fix emoji-failfast
  • Loading branch information
rusq authored Nov 10, 2023
2 parents 9670489 + ee42543 commit 49a1d5e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/app/emoji/emoji.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ type emojidumper interface {
DumpEmojis(ctx context.Context) (map[string]string, error)
}

func download(ctx context.Context, sess emojidumper, base string, ignoreErrors bool) error {
func download(ctx context.Context, sess emojidumper, base string, failFast bool) error {
fsa, err := fsadapter.New(base)
if err != nil {
return fmt.Errorf("unable to initialise adapter for %s: %w", base, err)
Expand All @@ -74,7 +74,7 @@ func download(ctx context.Context, sess emojidumper, base string, ignoreErrors b
return fmt.Errorf("failed writing emoji index: %w", err)
}

return fetch(ctx, fsa, emojis, true)
return fetch(ctx, fsa, emojis, failFast)
}

// fetch downloads the emojis and saves them to the fsa. It spawns numWorker
Expand Down

0 comments on commit 49a1d5e

Please sign in to comment.