Skip to content

Add UnnecessaryBackticks rule#4764

Merged
BraisGabin merged 1 commit intodetekt:mainfrom
t-kameyama:issue_4739
Apr 25, 2022
Merged

Add UnnecessaryBackticks rule#4764
BraisGabin merged 1 commit intodetekt:mainfrom
t-kameyama:issue_4739

Conversation

@t-kameyama
Copy link
Copy Markdown
Contributor

@t-kameyama t-kameyama commented Apr 25, 2022 •
edited
Loading

Fixes #4739

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 25, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
detekt ✅ Ready (Inspect) Visit Preview Apr 25, 2022 at 8:05AM (UTC)

@github-actions github-actions bot added the rules label Apr 25, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 25, 2022 •
edited
Loading

Codecov Report

Merging #4764 (5966655) into main (0380716) will increase coverage by 0.00%.
The diff coverage is 85.71%.

@@            Coverage Diff            @@
##               main    #4764   +/-   ##
=========================================
  Coverage     84.69%   84.69%           
- Complexity     3413     3425   +12     
=========================================
  Files           490      491    +1     
  Lines         11232    11253   +21     
  Branches       2065     2069    +4     
=========================================
+ Hits           9513     9531   +18     
- Misses          675      676    +1     
- Partials       1044     1046    +2
Impacted Files Coverage Δ
...urbosch/detekt/rules/style/UnnecessaryBackticks.kt 85.00% <85.00%> (ø)
...rturbosch/detekt/rules/style/StyleGuideProvider.kt 100.00% <100.00%> (ø)

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 0380716...5966655. Read the comment docs.

@t-kameyama t-kameyama marked this pull request as draft April 25, 2022 07:57
@t-kameyama t-kameyama marked this pull request as ready for review April 25, 2022 08:00
@cortinico cortinico added the notable changes Marker for notable changes in the changelog label Apr 25, 2022
@cortinico cortinico added this to the 1.21.0 milestone Apr 25, 2022
@BraisGabin BraisGabin merged commit f44947c into detekt:main Apr 25, 2022
@BraisGabin
Copy link
Copy Markdown
Member

BraisGabin commented Apr 25, 2022

Thanks!

@t-kameyama t-kameyama deleted the issue_4739 branch April 25, 2022 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BraisGabin BraisGabin BraisGabin approved these changes

@cortinico cortinico cortinico approved these changes

Assignees

No one assigned

Labels

notable changes Marker for notable changes in the changelog rules

Projects

None yet

Milestone

1.21.0

Development

Successfully merging this pull request may close these issues.

New rule: UnnecessaryBackticks

3 participants

@t-kameyama @BraisGabin @cortinico