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

New rule to check if @Preview (Jetpack Compose) functions end with 'Preview' suffix and are also private. Part 1 #1726

Merged
merged 6 commits into from
Sep 1, 2023

Conversation

kgevorkyan
Copy link
Member

@kgevorkyan kgevorkyan commented Aug 31, 2023

Part of #1322

What's done:

  • Provide warn logic
  • Add tests

### What's done:
* Wip
### What's done:
* Wip
### What's done:
* WIP
### What's done:
* WIP
@github-actions
Copy link
Contributor

github-actions bot commented Aug 31, 2023

JUnit Tests (macOS, EnricoMi/publish-unit-test-result-action@v2)

   163 files  +1     163 suites  +1   6m 34s ⏱️ - 2m 25s
1 375 tests +4  1 339 ✔️ +4  36 💤 ±0  0 ±0 
2 754 runs  +4  2 718 ✔️ +4  36 💤 ±0  0 ±0 

Results for commit 491724e. ± Comparison against base commit 6d7f952.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 31, 2023

JUnit Tests (Windows, EnricoMi/publish-unit-test-result-action@v2)

   163 files  +1     163 suites  +1   6m 39s ⏱️ - 4m 25s
1 375 tests +4  1 358 ✔️ +4  17 💤 ±0  0 ±0 
2 754 runs  +4  2 737 ✔️ +4  17 💤 ±0  0 ±0 

Results for commit 491724e. ± Comparison against base commit 6d7f952.

♻️ This comment has been updated with latest results.

### What's done:
* WIP
@codecov
Copy link

codecov bot commented Sep 1, 2023

Codecov Report

Merging #1726 (491724e) into master (6d7f952) will increase coverage by 0.02%.
The diff coverage is 82.50%.

@@             Coverage Diff              @@
##             master    #1726      +/-   ##
============================================
+ Coverage     78.20%   78.22%   +0.02%     
- Complexity     2312     2320       +8     
============================================
  Files           125      126       +1     
  Lines          8158     8198      +40     
  Branches       2061     2065       +4     
============================================
+ Hits           6380     6413      +33     
- Misses          874      877       +3     
- Partials        904      908       +4     
Files Changed Coverage Δ
...at/ruleset/rules/chapter3/PreviewAnnotationRule.kt 81.57% <81.57%> (ø)
...m/saveourtool/diktat/ruleset/constants/Warnings.kt 97.48% <100.00%> (+0.01%) ⬆️
...l/diktat/ruleset/rules/DiktatRuleSetFactoryImpl.kt 100.00% <100.00%> (ø)

### What's done:
* WIP
@kgevorkyan kgevorkyan changed the title [WIP] New rule to check if @Preview (Jetpack Compose) functions end with 'Preview' suffix and are also private New rule to check if @Preview (Jetpack Compose) functions end with 'Preview' suffix and are also private. Part 1 Sep 1, 2023
@kgevorkyan kgevorkyan marked this pull request as ready for review September 1, 2023 12:05
@kgevorkyan kgevorkyan merged commit fbf2a57 into master Sep 1, 2023
32 checks passed
@kgevorkyan kgevorkyan deleted the feature/preview_node branch September 1, 2023 12:58
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