Removed feature flag for PR locks (#110)

server/events/apply_requirement_handler.go

@@ -4,7 +4,6 @@ import (
 	"github.com/runatlantis/atlantis/server/events/models"
 	"github.com/runatlantis/atlantis/server/events/yaml/raw"
 	"github.com/runatlantis/atlantis/server/events/yaml/valid"
-	"github.com/runatlantis/atlantis/server/feature"
 )
 
 //go:generate pegomock generate -m --package mocks -o mocks/mock_apply_handler.go ApplyRequirement
@@ -13,8 +12,7 @@ type ApplyRequirement interface {
 }
 
 type AggregateApplyRequirements struct {
-	WorkingDir       WorkingDir
-	FeatureAllocator feature.Allocator
+	WorkingDir WorkingDir
 }
 
 func (a *AggregateApplyRequirements) ValidateProject(repoDir string, ctx models.ProjectCommandContext) (failure string, err error) {
@@ -38,12 +36,7 @@ func (a *AggregateApplyRequirements) ValidateProject(repoDir string, ctx models.
 				return "Default branch must be rebased onto pull request before running apply.", nil
 			}
 		case raw.UnlockedApplyRequirement:
-			shouldAllocate, err := a.FeatureAllocator.ShouldAllocate(feature.AtlantisLock, ctx.BaseRepo.FullName)
-			if err != nil {
-				ctx.Log.Err("unable to allocate for feature: %s, error: %s", feature.AtlantisLock, err)
-			}
-
-			if shouldAllocate && ctx.PullReqStatus.SQLocked {
+			if ctx.PullReqStatus.SQLocked {
 				return "Pull request must be unlocked using the 🔓  emoji before running apply.", nil
 			}
 		}

server/feature/names.go

@@ -4,4 +4,3 @@ type Name string
 
 // list of feature names used in the code base. These must be kept in sync with any external config.
 const LogStreaming Name = "log-streaming"
-const AtlantisLock Name = "atlantis-lock"

server/server.go

@@ -521,8 +521,7 @@ func NewServer(userConfig UserConfig, config Config) (*Server, error) {
 	}
 
 	applyRequirementHandler := &events.AggregateApplyRequirements{
-		WorkingDir:       workingDir,
-		FeatureAllocator: featureAllocator,
+		WorkingDir: workingDir,
 	}
 
 	projectCommandRunner := &events.DefaultProjectCommandRunner{