Use a maintained action for "changed files"

And replace "dummy" with "sample" in go tests to make `woke` happy :)

Signed-off-by: Vincent Demeester <[email protected]>

.github/workflows/woke.yml

@@ -7,16 +7,20 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: 'Checkout'
-        uses: actions/checkout@v2
-
-      - uses: jitterbit/get-changed-files@v1
-        id: files
+        uses: actions/checkout@v4
+
+      - name: Get changed files
+        id: changed-files
+        uses: tj-actions/changed-files@v40
+        with:
+          write_output_files: true
+          files: |
+            **
 
       - name: 'woke'
+        if: steps.changed-files.outputs.any_changed == 'true'
         uses: get-woke/woke-action@v0
         with:
           # Cause the check to fail on any broke rules
           fail-on-error: true
-          # See https://github.com/marketplace/actions/get-all-changed-files
-          # for more options
-          woke-args: ${{ steps.files.outputs.added_modified }}
+          woke-args: ${{ steps.changed_files.outputs.all_changed_files }}

.wokeignore

@@ -1,3 +1,5 @@
 go.mod
 go.sum
 vendor
+config/dummy.go
+.wokeignore

pkg/apis/pipeline/v1beta1/resource_types_test.go

@@ -21,13 +21,13 @@ import (
 var (
 	prependStep = v1beta1.Step{
 		Name:    "prepend-step",
-		Image:   "dummy",
+		Image:   "sample",
 		Command: []string{"doit"},
 		Args:    []string{"stuff", "things"},
 	}
 	appendStep = v1beta1.Step{
 		Name:    "append-step",
-		Image:   "dummy",
+		Image:   "sample",
 		Command: []string{"doit"},
 		Args:    []string{"other stuff", "other things"},
 	}

pkg/reconciler/pipelinerun/resources/pipelineref_test.go

@@ -53,9 +53,9 @@ import (
 )
 
 var (
-	dummyPipeline = &v1.Pipeline{
+	samplePipeline = &v1.Pipeline{
 		ObjectMeta: metav1.ObjectMeta{
-			Name:      "dummy",
+			Name:      "sample",
 			Namespace: "default",
 		},
 		TypeMeta: metav1.TypeMeta{
@@ -130,7 +130,7 @@ func TestLocalPipelineRef(t *testing.T) {
 	}{
 		{
 			name:      "local-pipeline",
-			pipelines: []runtime.Object{simplePipeline(), dummyPipeline},
+			pipelines: []runtime.Object{simplePipeline(), samplePipeline},
 			ref: &v1.PipelineRef{
 				Name: "simple",
 			},
@@ -191,9 +191,9 @@ func TestGetPipelineFunc_Local(t *testing.T) {
 		name: "local-pipeline",
 		localPipelines: []runtime.Object{
 			simplePipelineWithBaseSpec(),
-			dummyPipeline,
+			samplePipeline,
 		},
-		remotePipelines: []runtime.Object{simplePipeline(), dummyPipeline},
+		remotePipelines: []runtime.Object{simplePipeline(), samplePipeline},
 		ref: &v1.PipelineRef{
 			Name: "simple",
 		},
@@ -240,7 +240,7 @@ func TestGetPipelineFuncSpecAlreadyFetched(t *testing.T) {
 	ctx := context.Background()
 	ctx, cancel := context.WithCancel(ctx)
 	defer cancel()
-	tektonclient := fake.NewSimpleClientset(simplePipeline(), dummyPipeline)
+	tektonclient := fake.NewSimpleClientset(simplePipeline(), samplePipeline)
 	kubeclient := fakek8s.NewSimpleClientset(&corev1.ServiceAccount{
 		ObjectMeta: metav1.ObjectMeta{
 			Namespace: "default",