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

ci(golangci): update config #276

Merged
merged 2 commits into from
Apr 16, 2024
Merged

Conversation

sambacha
Copy link
Contributor

Remove usage of depreciated values

Description

makea da golangci config all nice and good

Tests

i run locally it nice and good

Additional context

maybe add output config for nice good results ya

# output configuration options
output:
  # The formats used to render issues.
  # Format: `colored-line-number`, `line-number`, `json`, `colored-tab`, `tab`, `checkstyle`, `code-climate`, `junit-xml`, `github-actions`, `teamcity`
  # Output path can be either `stdout`, `stderr` or path to the file to write to.
  #
  # For the CLI flag (`--out-format`), multiple formats can be specified by separating them by comma.
  # The output can be specified for each of them by separating format name and path by colon symbol.
  # Example: "--out-format=checkstyle:report.xml,json:stdout,colored-line-number"
  # The CLI flag (`--out-format`) override the configuration file.
  #
  # Default:
  #   formats:
  #     - format: colored-line-number
  #       path: stdout
  formats:
    - format: json
      path: stderr
    - format: checkstyle
      path: report.xml
    - format: colored-line-number

Metadata

  • Fixes #[Link to Issue]

Remove usage of depreciated values
@sambacha sambacha requested a review from a team as a code owner March 29, 2024 13:28
@sambacha sambacha requested review from ajsutton and removed request for a team March 29, 2024 13:28
@ajsutton ajsutton merged commit b642a5b into ethereum-optimism:optimism Apr 16, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants