Linting_tests (#34)

* mounting a volume to buildkit

* squash last 12 commits

* remove toml

* remove volume

* retrigger tests

* making import export dynamic

* set the startup flag

* Add tests support for startup & refactoring

* testing

* testing

* testing

* testing

* testing

* testing

* testing

* testing

* testing

* testing

* was sh all along

* fix startup string to bool

* removing ts for testing as building it alone fails

* runtime error while building c

* now go failed

* rust failed

* just py,rb,node

* just py,rb,node

* trigger all the tests

* trigger all the tests

* trigger all the tests

* trigger all the tests

* trigger all the tests

* trigger all the tests startup with cli, i am soooo dumb

* i think this might work now

* final testing

* retrigger

* what happens if i trigger tests with only runtime build

* retrigger with backtrace added

* add diff registry for run time/dev to isolate and retrigger tests with both open

* disable backtrace like before, this test should fail

* triggering only the startup tests with backtrace enabled

* enable backtrace,both issues resolved, very demure, very mindful

* cleanup and enable all tests

* only add startup-rootless and startup-client as tests

* Seperated tests

* linting tests

* fix lint

.github/workflows/golangci-lint.yml

@@ -0,0 +1,30 @@
+name: golangci-lint
+on:
+  push:
+    branches:
+      - main
+      - master
+  pull_request:
+
+permissions:
+  contents: read
+  # Optional: allow read access to pull request. Use with `only-new-issues` option.
+  pull-requests: read
+
+jobs:
+  golangci:
+    strategy:
+      matrix:
+        go: [stable]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    name: lint
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go }}
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.60

pkg/staging/helper.go

@@ -41,22 +41,22 @@ func copy(src llb.State, srcPath string, dest llb.State, destPath string) llb.St
 	}))
 }
 
-func copyForStates(src llb.State, dst llb.State, srcpaths []string, dstpath string) llb.State {
-	return dst.With(
-		copyMultiple(src, srcpaths, dstpath),
-	)
-}
-
-func copyMultiple(src llb.State, srcPaths []string, destPath string) llb.StateOption {
-	var stateOptions []llb.StateOption
-	for _, srcPath := range srcPaths {
-		stateOptions = append(stateOptions, copyFrom(src, srcPath, destPath))
-	}
-
-	return func(s llb.State) llb.State {
-		for _, stateOption := range stateOptions {
-			s = stateOption(s)
-		}
-		return s
-	}
-}
+// Functions which might be useful later, left as comments here : 
+// func copyForStates(src llb.State, dst llb.State, srcpaths []string, dstpath string) llb.State {
+// 	return dst.With(
+// 		copyMultiple(src, srcpaths, dstpath),
+// 	)
+// }
+
+// func copyMultiple(src llb.State, srcPaths []string, destPath string) llb.StateOption {
+// 	var stateOptions []llb.StateOption
+// 	for _, srcPath := range srcPaths {
+// 		stateOptions = append(stateOptions, copyFrom(src, srcPath, destPath))
+// 	}
+// 	return func(s llb.State) llb.State {
+// 		for _, stateOption := range stateOptions {
+// 			s = stateOption(s)
+// 		}
+// 		return s
+// 	}
+// }