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

chore: linter settings for nats-manager #92

Merged
merged 5 commits into from
Feb 28, 2024

Conversation

the1bit
Copy link
Contributor

@the1bit the1bit commented Feb 28, 2024

Description

Changes proposed in this pull request:

Linter settings:

  • importas
  • gomoddirectives
  • perfsprint
  • tagalign
  • thelper
  • dupword
  • funlen
  • gci
  • bodyclose
  • gofumpt
  • goerr113
  • godox
  • goconst

Related issue(s)

@the1bit the1bit requested a review from a team as a code owner February 28, 2024 14:41
@the1bit the1bit requested a review from mfaizanse February 28, 2024 14:41
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 28, 2024
@the1bit the1bit added the area/ci Issues or PRs related to CI related topics label Feb 28, 2024
@the1bit the1bit changed the title chore: Linter settings for nats-manager chore: linter settings for nats-manager Feb 28, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 28, 2024
@kyma-bot kyma-bot merged commit abb93fa into kyma-project:main Feb 28, 2024
6 of 7 checks passed
@the1bit the1bit deleted the Lint-NATS-Manager branch February 28, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues or PRs related to CI related topics cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants