Skip to content

Make linting work with remote golangci-lint config #1522

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.

Sign up for GitHub

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

Jump to bottom
Merged
ilija42 merged 3 commits into from
Sep 3, 2025
Merged

Make linting work with remote golangci-lint config #1522

ilija42 merged 3 commits into from
Sep 3, 2025

Conversation

@chainchad
Copy link
Contributor

@chainchad chainchad commented Sep 3, 2025

  • Remove local golangci-lint config
  • Make linting work with remote config

github-advanced-security[bot]
github-advanced-security bot found potential problems Sep 3, 2025
View reviewed changes
@github-actions
Copy link

github-actions bot commented Sep 3, 2025 •
edited
Loading

apidiff results - no incompatible changes detected ✅

(Full summary: https://github.com/smartcontractkit/chainlink-common/actions/runs/17442066202/#summary-49527299110)

@chainchad chainchad marked this pull request as ready for review September 3, 2025 17:51
@chainchad chainchad requested a review from a team as a code owner September 3, 2025 17:51
@ilija42 ilija42 enabled auto-merge (squash) September 3, 2025 18:05
@ilija42 ilija42 merged commit 6a3e476 into main Sep 3, 2025
19 of 20 checks passed
@ilija42 ilija42 deleted the dx-1521/golangci-lint-composite-action branch September 3, 2025 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmank88 jmank88 jmank88 approved these changes

@ilija42 ilija42 ilija42 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chainchad @jmank88 @ilija42