Use testscript for CLI test #334
Annotations
10 errors and 1 warning
CLI Test
ginkgo-linter: multiple issues: timeout defined more than once; polling defined more than once
|
CLI Test
ginkgo-linter: multiple issues: only use time.Duration for timeout and polling in Eventually() or Consistently(); only use time.Duration for timeout and polling in Eventually() or Consistently(). Consider using `Consistently(func() bool { return true }, time.Second*10, time.Second).Should(BeTrue())` instead
|
CLI Test
ginkgo-linter: timeout must not be shorter than the polling interval
|
CLI Test
ginkgo-linter: Success matcher only support a single error value, or function with Gomega as its first parameter
|
CLI Test
ginkgo-linter: wrong error assertion. Consider using `Eventually(func() error { return err }).Should(Succeed())` instead
|
CLI Test
ginkgo-linter: Success matcher does not support multiple values
|
CLI Test
ginkgo-linter: Success matcher only support a single error value, or function with Gomega as its first parameter
|
CLI Test
ginkgo-linter: wrong error assertion. Consider using `Eventually(func() error { return err }).Should(Succeed())` instead
|
CLI Test
ginkgo-linter: Success matcher does not support multiple values
|
CLI Test
ginkgo-linter: prefer using the HaveOccurred matcher for non-function error value, instead of Succeed. Consider using `Expect(err).To(HaveOccurred())` instead
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, 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/
|
Loading