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

Bump spotbugs from 4.6.0 to 4.7.1 #914

Closed
wants to merge 1 commit into from
Closed

Conversation

shalk
Copy link
Contributor

@shalk shalk commented Jun 28, 2022

relate to spotbugs/spotbugs#2040 fixed in 4.7.1

@codecov-commenter
Copy link

Codecov Report

Merging #914 (43cb664) into master (01c61f9) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #914      +/-   ##
============================================
- Coverage     91.73%   91.72%   -0.02%     
+ Complexity     7437     7436       -1     
============================================
  Files           188      188              
  Lines         15945    15945              
  Branches       2994     2994              
============================================
- Hits          14627    14625       -2     
- Misses          725      726       +1     
- Partials        593      594       +1     
Impacted Files Coverage Δ
...va/org/apache/commons/lang3/CharSequenceUtils.java 87.09% <0.00%> (-1.62%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 01c61f9...43cb664. Read the comment docs.

@garydgregory
Copy link
Member

Hi @shalk
Unless the update needs code changes, there is no need for this type of PR as we use Dependabot.

@shalk
Copy link
Contributor Author

shalk commented Jun 28, 2022

ok, i think so.

@shalk shalk closed this Jun 28, 2022
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.

3 participants