Skip to content

Conversation

@EndBug
Copy link
Member

@EndBug EndBug commented Jun 30, 2022

Closes #462, closes #465

Changes were proposed in those PRs but were not organized and weren't passing tests. This is cleaner.

@EndBug EndBug added the type: feature New feature or feature request label Jun 30, 2022
@EndBug EndBug requested a review from Snazzah June 30, 2022 11:43
@EndBug EndBug self-assigned this Jun 30, 2022
@codecov
Copy link

codecov bot commented Jun 30, 2022

Codecov Report

Merging #466 (710caff) into master (adb1219) will increase coverage by 0.05%.
The diff coverage is 100.00%.

❗ Current head 710caff differs from pull request most recent head a0ffe49. Consider uploading reports for the commit a0ffe49 to get more accurate results

@@            Coverage Diff             @@
##           master     #466      +/-   ##
==========================================
+ Coverage   97.50%   97.55%   +0.05%     
==========================================
  Files          41       42       +1     
  Lines        1161     1186      +25     
  Branches      199      202       +3     
==========================================
+ Hits         1132     1157      +25     
  Misses         29       29              
Impacted Files Coverage Δ
src/Interface/ListIndex.ts 100.00% <100.00%> (ø)
src/Interface/Lists/RadarBotList.ts 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 adb1219...a0ffe49. Read the comment docs.

@EndBug EndBug merged commit 7466275 into master Jun 30, 2022
@EndBug EndBug deleted the radarbotlist branch June 30, 2022 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature New feature or feature request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants