move the isLocal check to Scope, and remove the arg from the constructor

HaxeFoundation · Jul 30, 2024 · 508d461 · 508d461
1 parent d9a4b88
commit 508d461
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/run.n b/run.n
diff --git a/src/haxelib/api/GlobalScope.hx b/src/haxelib/api/GlobalScope.hx
@@ -22,10 +22,7 @@ using StringTools;
 **/
 class GlobalScope extends Scope {
 	function new(repository:Repository) {
-		super(false, repository);
-
-		// While this class is "GlobalScope", there's currently not a "LocalScope" equivalent class. For now we can add this here. 
-		isLocal = repository.path != RepoManager.suggestGlobalPath() ?? "";
+		super(repository);
 	}
 
 	public function runScript(library:ProjectName, ?callData:CallData, ?version:Version):Void {

diff --git a/src/haxelib/api/Scope.hx b/src/haxelib/api/Scope.hx
@@ -60,9 +60,10 @@ abstract class Scope {
 	public final repository:Repository;
 	final overrides:LockFormat;
 
-	function new(isLocal:Bool, repository:Repository) {
-		this.isLocal = isLocal;
+	function new(repository:Repository) {
 		this.repository = repository;
+		// While this class is "GlobalScope", there's currently not a "LocalScope" equivalent class. For now we can add this here. 
+		isLocal = repository.path != RepoManager.suggestGlobalPath() ?? "";
 
 		overrides = loadOverrides();
 	}