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

Run optional linters only for pull requests #1280

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Run optional linters only for pull requests #1280

merged 1 commit into from
Oct 4, 2022

Conversation

artemgavrilov
Copy link
Contributor

Disable optional linters run until this issue will be resolved: reviewdog/action-golangci-lint#432

@codecov
Copy link

codecov bot commented Oct 4, 2022

Codecov Report

Merging #1280 (eed9987) into main (49a4a13) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1280      +/-   ##
==========================================
- Coverage   44.10%   44.05%   -0.05%     
==========================================
  Files         334      334              
  Lines       39777    39777              
==========================================
- Hits        17542    17525      -17     
- Misses      20711    20727      +16     
- Partials     1524     1525       +1     
Flag Coverage Δ
admin 8.50% <ø> (ø)
agent 53.86% <ø> (-0.18%) ⬇️
managed 44.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
agent/runner/runner.go 72.45% <0.00%> (-8.99%) ⬇️
.../agents/mongodb/internal/profiler/sender/sender.go 79.03% <0.00%> (-3.23%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@artemgavrilov artemgavrilov marked this pull request as ready for review October 4, 2022 11:33
@artemgavrilov artemgavrilov requested review from BupycHuk and a team as code owners October 4, 2022 11:33
@BupycHuk BupycHuk merged commit dcaf639 into main Oct 4, 2022
@BupycHuk BupycHuk deleted the fix-reviewdog branch October 4, 2022 11:54
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.

Action crashes when golangci-lint output is big
2 participants