feat: add refetch option to remotes config

evilmartians · May 31, 2024 · e1d8aef · e1d8aef
1 parent c2b2db7
commit e1d8aef
Show file tree

Hide file tree

Showing 5 changed files with 27 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -20,7 +20,7 @@ bench:
 
 bin/golangci-lint:
 	@test -x $$(go env GOPATH)/bin/golangci-lint || \
-		curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $$(go env GOPATH)/bin v1.56.2
+		curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $$(go env GOPATH)/bin v1.59.0
 
 lint: bin/golangci-lint
 	$$(go env GOPATH)/bin/golangci-lint run --fix

diff --git a/internal/config/remote.go b/internal/config/remote.go
@@ -6,6 +6,7 @@ type Remote struct {
 	// Deprecated
 	Config  string   `json:"config,omitempty"  mapstructure:"config,omitempty"  toml:"config,omitempty"  yaml:",omitempty"`
 	Configs []string `json:"configs,omitempty" mapstructure:"configs,omitempty" toml:"configs,omitempty" yaml:",omitempty"`
+	Refetch bool     `json:"refetch,omitempty" mapstructure:"refetch,omitempty" toml:"refetch,omitempty" yaml:",omitempty"`
 }
 
 func (r *Remote) Configured() bool {

diff --git a/internal/git/remote.go b/internal/git/remote.go
@@ -39,6 +39,11 @@ func (r *Repository) SyncRemote(url, ref string, force bool) error {
 		return err
 	}
 
+	log.SetName("fetching remotes")
+	log.StartSpinner()
+	defer log.StopSpinner()
+	defer log.UnsetName("fetching remotes")
+
 	directoryName := remoteDirectoryName(url, ref)
 	remotePath := filepath.Join(remotesPath, directoryName)
 

diff --git a/internal/lefthook/install.go b/internal/lefthook/install.go
@@ -108,6 +108,25 @@ func (l *Lefthook) createConfig(path string) error {
 	return nil
 }
 
+func (l *Lefthook) syncHooks(cfg *config.Config, checkHashSum, force bool) error {
+	for _, remote := range cfg.Remotes {
+		if remote.Configured() && remote.Refetch {
+			if err := l.repo.SyncRemote(remote.GitURL, remote.Ref, force); err != nil {
+				checkHashSum = false
+				log.Warnf("Couldn't sync remotes. Will continue without them: %s", err)
+			} else {
+				// Reread the config file with synced remotes
+				cfg, err = l.readOrCreateConfig()
+				if err != nil {
+					return err
+				}
+			}
+		}
+	}
+
+	return l.createHooksIfNeeded(cfg, checkHashSum, force)
+}
+
 func (l *Lefthook) createHooksIfNeeded(cfg *config.Config, checkHashSum, force bool) error {
 	if checkHashSum && l.hooksSynchronized() {
 		return nil

diff --git a/internal/lefthook/run.go b/internal/lefthook/run.go
@@ -97,7 +97,7 @@ func (l *Lefthook) Run(hookName string, args RunArgs, gitArgs []string) error {
 
 	if !args.NoAutoInstall {
 		// This line controls updating the git hook if config has changed
-		if err = l.createHooksIfNeeded(cfg, true, false); err != nil {
+		if err = l.syncHooks(cfg, true, false); err != nil {
 			log.Warn(
 				`⚠️  There was a problem with synchronizing git hooks.
 Run 'lefthook install' manually.`,