Skip to content

General improvements and clean up#177

Merged
VJalili merged 1 commit intoGenometric:devfrom
VJalili:code-smell
Oct 16, 2022
Merged

General improvements and clean up#177
VJalili merged 1 commit intoGenometric:devfrom
VJalili:code-smell

Conversation

@VJalili
Copy link
Copy Markdown
Member

@VJalili VJalili commented Oct 16, 2022

  • Code smell fixes;
  • Improve readability (refactor string interpolations, var declarations, array join, line breaks);
  • Remove license header.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 16, 2022

Codecov Report

Merging #177 (e4816dc) into dev (0689233) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #177      +/-   ##
==========================================
- Coverage   97.13%   97.12%   -0.02%     
==========================================
  Files          43       43              
  Lines        2167     2156      -11     
==========================================
- Hits         2105     2094      -11     
  Misses         13       13              
  Partials       49       49              
Impacted Files Coverage Δ
Core.Tests/Basic/AdjPValue.cs 100.00% <ø> (ø)
Core.Tests/Basic/C.cs 100.00% <ø> (ø)
Core.Tests/Basic/ColocalizationCount.cs 100.00% <ø> (ø)
Core.Tests/Basic/CompareProcessedPeaks.cs 100.00% <ø> (ø)
Core.Tests/Basic/MultipleIntersections.cs 100.00% <ø> (ø)
Core.Tests/Basic/PeakReason.cs 100.00% <ø> (ø)
Core.Tests/Basic/TConsensusPeaks.cs 90.54% <ø> (ø)
Core.Tests/Basic/TPeak.cs 100.00% <ø> (ø)
Core.Tests/Basic/TSupportingPeak.cs 100.00% <ø> (ø)
Core.Tests/Concurrency/Concurrency.cs 100.00% <ø> (ø)
... and 33 more

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

@VJalili VJalili merged commit ad233fa into Genometric:dev Oct 16, 2022
@VJalili VJalili deleted the code-smell branch October 16, 2022 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant