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

Fixed PreviewAnnotationRule #1740

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Fixed PreviewAnnotationRule #1740

merged 1 commit into from
Sep 21, 2023

Conversation

nulls
Copy link
Member

@nulls nulls commented Sep 21, 2023

What's done:

  • moved creating private node to ASTFactory
  • added a new node to ANNOTATION_LIST

It's part of #1737

### What's done:
- moved creating private node to ASTFactory
- added a new node to ANNOTATION_LIST

It's part of 1737
@nulls nulls added bug Something isn't working tests Improve testing labels Sep 21, 2023
@nulls nulls added this to the 2.0.0 milestone Sep 21, 2023
@nulls nulls self-assigned this Sep 21, 2023
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #1740 (0083d1c) into master (11ea0aa) will increase coverage by 0.23%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##             master    #1740      +/-   ##
============================================
+ Coverage     74.76%   75.00%   +0.23%     
- Complexity     2270     2274       +4     
============================================
  Files           126      126              
  Lines          8230     8233       +3     
  Branches       2072     2066       -6     
============================================
+ Hits           6153     6175      +22     
+ Misses         1163     1142      -21     
- Partials        914      916       +2     
Files Changed Coverage Δ
...at/ruleset/rules/chapter3/PreviewAnnotationRule.kt 87.87% <85.71%> (+30.73%) ⬆️

@github-actions
Copy link
Contributor

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

   164 files  ±0     164 suites  ±0   6m 40s ⏱️ + 1m 55s
1 377 tests ±0  1 309 ✔️ +1  68 💤  - 1  0 ±0 
2 756 runs  ±0  2 687 ✔️ +1  69 💤  - 1  0 ±0 

Results for commit 0083d1c. ± Comparison against base commit 11ea0aa.

@github-actions
Copy link
Contributor

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

   164 files  ±0     164 suites  ±0   7m 51s ⏱️ - 1m 30s
1 377 tests ±0  1 321 ✔️ +1  56 💤  - 1  0 ±0 
2 756 runs  ±0  2 699 ✔️ +1  57 💤  - 1  0 ±0 

Results for commit 0083d1c. ± Comparison against base commit 11ea0aa.

@nulls nulls enabled auto-merge (squash) September 21, 2023 09:22
@nulls nulls merged commit dd5542b into master Sep 21, 2023
33 checks passed
@nulls nulls deleted the feature/fix-tests#1737 branch September 21, 2023 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tests Improve testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants