feat: [CODE-2324]: added reviewer-add PR activity entry (#2698) #959
bot2-harnessci-lint.yml
on: push
CI linter for js/ts
3m 49s
CI linter for go
45s
Annotations
10 errors and 24 warnings
|
CI linter for go:
registry/app/storage/blobwriter.go#L39
G101: Potential hardcoded credentials (gosec)
|
|
CI linter for go:
registry/app/storage/gcstoragelient.go#L46
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
registry/app/driver/s3-aws/s3.go#L1196
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
types/enum/gitspace_event_type.go#L82
G101: Potential hardcoded credentials (gosec)
|
|
CI linter for go:
app/services/label/label.go#L153
G601: Implicit memory aliasing in for loop. (gosec)
|
|
CI linter for go:
app/services/label/label.go#L165
G601: Implicit memory aliasing in for loop. (gosec)
|
|
CI linter for go:
app/services/label/label.go#L166
G601: Implicit memory aliasing in for loop. (gosec)
|
|
CI linter for go:
app/services/migrate/pullreq.go#L108
G601: Implicit memory aliasing in for loop. (gosec)
|
|
CI linter for go:
app/services/migrate/pullreq.go#L362
G601: Implicit memory aliasing in for loop. (gosec)
|
|
CI linter for go:
app/store/database/label_pullreq.go#L179
G601: Implicit memory aliasing in for loop. (gosec)
|
|
CI linter for go
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
CI linter for go:
types/label.go#L150
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
types/label.go#L175
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
types/label.go#L192
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
types/label.go#L211
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
app/services/infraprovider/update.go#L68
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
app/services/infraprovider/update.go#L91
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
app/api/controller/infraprovider/create.go#L110
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
registry/app/pkg/docker/local.go#L1635
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
registry/app/pkg/docker/local.go#L1675
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go:
registry/app/pkg/docker/local.go#L1738
if-return: redundant if ...; err != nil check, just return error instead. (revive)
|
|
CI linter for go
The process '/usr/bin/git' failed with exit code 128
|
|
CI linter for js/ts
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
CI linter for js/ts:
web/src/AppContext.tsx#L86
React Hook useEffect has a missing dependency: 'initialValue.isCurrentSessionPublic'. Either include it or remove the dependency array
|
|
CI linter for js/ts:
web/src/ar/MFEAppTypes.ts#L74
Unexpected any. Specify a different type
|
|
CI linter for js/ts:
web/src/ar/__mocks__/contexts/PreferenceStoreContext.tsx#L43
Unexpected any. Specify a different type
|
|
CI linter for js/ts:
web/src/ar/__mocks__/contexts/PreferenceStoreContext.tsx#L93
Unexpected any. Specify a different type
|
|
CI linter for js/ts:
web/src/ar/__mocks__/hooks/useConfirmationDialog.tsx#L90
React Hook React.useCallback has a missing dependency: 'showModal'. Either include it or remove the dependency array
|
|
CI linter for js/ts:
web/src/ar/__mocks__/hooks/useQueryParams.ts#L24
Unexpected any. Specify a different type
|
|
CI linter for js/ts:
web/src/ar/__mocks__/hooks/useQueryParams.ts#L119
React Hook useMemo has a missing dependency: 'decoderOptions'. Either include it or remove the dependency array
|
|
CI linter for js/ts:
web/src/ar/app/App.tsx#L88
React Hook useEffect has missing dependencies: 'appStoreData' and 'parent'. Either include them or remove the dependency array
|
|
CI linter for js/ts:
web/src/ar/app/GitnessApp.tsx#L68
Unexpected any. Specify a different type
|
|
CI linter for js/ts:
web/src/ar/app/GitnessApp.tsx#L69
Unexpected any. Specify a different type
|
|
CI linter for js/ts
The process '/usr/bin/git' failed with exit code 128
|