Skip to content

Use coerceArray#418

Merged
groenroos merged 1 commit intomasterfrom
fix/use-array-coercer
Feb 21, 2022
Merged

Use coerceArray#418
groenroos merged 1 commit intomasterfrom
fix/use-array-coercer

Conversation

@groenroos
Copy link
Member

#416 introduced a coerceArray method for forcing any variable into an array.

This PR implements that method throughout the app.

@groenroos groenroos added the refactoring Drastic code quality improvements label Feb 21, 2022
@groenroos groenroos added this to the M2 - Release 2.0 milestone Feb 21, 2022
@groenroos groenroos self-assigned this Feb 21, 2022
@codecov
Copy link

codecov bot commented Feb 21, 2022

Codecov Report

Merging #418 (643a5bb) into master (1cb6ee1) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #418      +/-   ##
==========================================
- Coverage   98.05%   98.05%   -0.01%     
==========================================
  Files          37       37              
  Lines        4581     4575       -6     
==========================================
- Hits         4492     4486       -6     
  Misses         89       89              
Impacted Files Coverage Δ
core/loadServer.js 94.48% <100.00%> (-0.04%) ⬇️
drivers/db/Memory.js 99.09% <100.00%> (-0.03%) ⬇️
lib/Response.js 98.45% <100.00%> (-0.01%) ⬇️
lib/User.js 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 1cb6ee1...643a5bb. Read the comment docs.

@groenroos groenroos merged commit e3c9ca5 into master Feb 21, 2022
@groenroos groenroos deleted the fix/use-array-coercer branch February 21, 2022 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Drastic code quality improvements

Development

Successfully merging this pull request may close these issues.

1 participant