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

Address Code Style Violations #2102

Merged
merged 6 commits into from
Jun 5, 2023
Merged

Conversation

Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Jun 2, 2023

Brings us down from 8k to 1.4k code violations

  • cleanup usings
  • mark embedded libraries as generated code to reduce ammount of code style violations
  • apply braces style
  • prefer discard parameter
  • default value initializing
  • address more code violations

@elastic-apm-tech elastic-apm-tech added this to In Progress in APM-Agents (OLD) Jun 2, 2023
@Mpdreamz Mpdreamz merged commit 2156fb7 into main Jun 5, 2023
13 checks passed
APM-Agents (OLD) automation moved this from In Progress to Done Jun 5, 2023
@Mpdreamz Mpdreamz deleted the fix/address-code-analyzer-issues branch June 5, 2023 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant