Skip to content

Conversation

@ascopes
Copy link
Owner

@ascopes ascopes commented Oct 22, 2022

No description provided.

@ascopes ascopes force-pushed the bugfix/win32-path-regression branch from 1f3edf6 to 52bc97f Compare October 22, 2022 10:17
@sonatype-lift
Copy link

sonatype-lift bot commented Oct 22, 2022

⚠️ 3 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

@codecov-commenter
Copy link

codecov-commenter commented Oct 22, 2022

Codecov Report

Merging #110 (52bc97f) into main (2b83729) will decrease coverage by 0.13%.
The diff coverage is 36.84%.

@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   53.14%   53.01%   -0.14%     
==========================================
  Files          69       69              
  Lines        2510     2522      +12     
  Branches      226      230       +4     
==========================================
+ Hits         1334     1337       +3     
- Misses       1083     1090       +7     
- Partials       93       95       +2     
Impacted Files Coverage Δ
...thub/ascopes/jct/compilers/FileManagerBuilder.java 50.00% <33.33%> (-0.88%) ⬇️
...github/ascopes/jct/pathwrappers/RamFileSystem.java 42.37% <37.50%> (-1.15%) ⬇️

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

@github-actions
Copy link

Test Results

   385 files  ±0     385 suites  ±0   52m 58s ⏱️ - 3m 29s
8 835 tests ±0  8 751 ✔️ +21  84 💤 ±0  0  - 21 
8 989 runs  ±0  8 905 ✔️ +21  84 💤 ±0  0  - 21 

Results for commit 52bc97f. ± Comparison against base commit 2b83729.

@ascopes
Copy link
Owner Author

ascopes commented Oct 22, 2022

warning 3 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

@sonatype-lift ignore

@ascopes ascopes merged commit b3e4c58 into main Oct 22, 2022
@ascopes ascopes deleted the bugfix/win32-path-regression branch October 25, 2022 06:17
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