Skip to content

Commit

Permalink
Add nolint: golint to conversion file 👃
Browse files Browse the repository at this point in the history
The main reason is to not run into error forcing us to use the same
receiver name for all function. The receiver name are different for a
good reason.

Signed-off-by: Vincent Demeester <[email protected]>
  • Loading branch information
vdemeester authored and tekton-robot committed Feb 11, 2020
1 parent a38a26b commit 94d18e6
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 2 deletions.
1 change: 1 addition & 0 deletions pkg/apis/pipeline/v1alpha1/cluster_task_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

// nolint: golint
package v1alpha1

import (
Expand Down
1 change: 1 addition & 0 deletions pkg/apis/pipeline/v1alpha1/pipeline_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

// nolint: golint
package v1alpha1

import (
Expand Down
1 change: 1 addition & 0 deletions pkg/apis/pipeline/v1alpha1/task_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

// nolint: golint
package v1alpha1

import (
Expand Down
1 change: 1 addition & 0 deletions pkg/apis/pipeline/v1alpha2/cluster_task_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

// nolint: golint
package v1alpha2

import (
Expand Down
1 change: 1 addition & 0 deletions pkg/apis/pipeline/v1alpha2/pipeline_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

// nolint: golint
package v1alpha2

import (
Expand Down
1 change: 1 addition & 0 deletions pkg/apis/pipeline/v1alpha2/task_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

// nolint: golint
package v1alpha2

import (
Expand Down
4 changes: 2 additions & 2 deletions pkg/reconciler/taskrun/resources/apply_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -730,7 +730,7 @@ func TestApplyWorkspaces(t *testing.T) {

func TestTaskResults(t *testing.T) {
names.TestingSeed()
ts := &v1alpha1.TaskSpec{
ts := &v1alpha1.TaskSpec{TaskSpec: v1alpha2.TaskSpec{
Results: []v1alpha1.TaskResult{{
Name: "current-date-unix-timestamp",
Description: "The current date in unix timestamp format",
Expand All @@ -752,7 +752,7 @@ func TestTaskResults(t *testing.T) {
},
Script: "#!/usr/bin/env bash\ndate | tee $(results.current-date-human-readable.path)",
}},
}
}}
want := applyMutation(ts, func(spec *v1alpha1.TaskSpec) {
spec.Steps[0].Script = "#!/usr/bin/env bash\ndate +%s | tee /tekton/results/current-date-unix-timestamp"
spec.Steps[0].Args[0] = "/tekton/results/current-date-unix-timestamp"
Expand Down

0 comments on commit 94d18e6

Please sign in to comment.