Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spotless should have taken care of groovy source code #12143

Open
heyams opened this issue Aug 29, 2024 · 1 comment
Open

Spotless should have taken care of groovy source code #12143

heyams opened this issue Aug 29, 2024 · 1 comment
Labels
enhancement New feature or request needs triage New issue that requires triage

Comments

@heyams
Copy link
Contributor

heyams commented Aug 29, 2024

Is your feature request related to a problem? Please describe.

build failed because of Task :instrumentation:couchbase:couchbase-common:testing:codenarcMain FAILED

steps to resolve:

  • download huge CI log
  • search for 'Failed` from the bottom up
  • no helpful message there
  • ran the task locally
  • offer a link to check codenarc report

found this

image

Describe the solution you'd like

`./gradlew spotlessApply' should have taken care of it.

Describe alternatives you've considered

  • fix it manually one by one based on the CodeNarc report. it's time consuming and needs to be automated.
  • convert all the groovy classes to java (which is on-going) :)

Additional context

No response

@heyams heyams added enhancement New feature or request needs triage New issue that requires triage labels Aug 29, 2024
@heyams
Copy link
Contributor Author

heyams commented Aug 29, 2024

Looks like it's supported already.. not sure why it didn't work for groovy class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs triage New issue that requires triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant